diff options
| author | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-27 11:13:42 +0000 |
|---|---|---|
| committer | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-27 11:13:42 +0000 |
| commit | 80ea2b5ffd514de5507ed9df550cbb6c57e4e94b (patch) | |
| tree | 693558e08f9bd620e9b3b7a6f72d6efafb20298c | |
| parent | ae328d6f152b5c95934791ce3bbe08b1ab40ea77 (diff) | |
| download | chromium_src-80ea2b5ffd514de5507ed9df550cbb6c57e4e94b.zip chromium_src-80ea2b5ffd514de5507ed9df550cbb6c57e4e94b.tar.gz chromium_src-80ea2b5ffd514de5507ed9df550cbb6c57e4e94b.tar.bz2 | |
Extension App provider should include packaged apps as well. They have no
web launch URL (which is why they don't show up right now), but they have
a local extension url we can launch.
BUG=80504
TEST=Install a packaged app, such as SparkChess, and make sure it appears in Omnibox when you type in SparkChess.
Review URL: http://codereview.chromium.org/6901025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83139 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/autocomplete/extension_app_provider.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/autocomplete/extension_app_provider.cc b/chrome/browser/autocomplete/extension_app_provider.cc index 4c61e0d..614cb1f 100644 --- a/chrome/browser/autocomplete/extension_app_provider.cc +++ b/chrome/browser/autocomplete/extension_app_provider.cc @@ -94,9 +94,10 @@ void ExtensionAppProvider::RefreshAppList() { extension_apps_.clear(); for (ExtensionList::const_iterator app = extensions->begin(); app != extensions->end(); ++app) { - if ((*app)->is_app() && !(*app)->launch_web_url().empty()) { - extension_apps_.push_back(std::make_pair((*app)->name(), - (*app)->launch_web_url())); + if ((*app)->is_app() && (*app)->GetFullLaunchURL().is_valid()) { + extension_apps_.push_back( + std::make_pair((*app)->name(), + (*app)->GetFullLaunchURL().spec())); } } } |