| Index: chrome/browser/ui/browser_init.cc
|
| diff --git a/chrome/browser/ui/browser_init.cc b/chrome/browser/ui/browser_init.cc
|
| index 84c85fffb905b6f8cf418fe155535130c9826f99..b2a18d8685197e9296ea7d176267964e17213eb1 100644
|
| --- a/chrome/browser/ui/browser_init.cc
|
| +++ b/chrome/browser/ui/browser_init.cc
|
| @@ -604,14 +604,9 @@ bool BrowserInit::LaunchWithProfile::OpenApplicationWindow(Profile* profile) {
|
| if (!IsAppLaunch(&url_string, &app_id))
|
| return false;
|
|
|
| - // http://crbug.com/37548
|
| - // TODO(rafaelw): There are two legitimate cases where the extensions
|
| - // service could not be ready at this point which need to be handled:
|
| - // 1) The locale has changed and the manifests stored in the preferences
|
| - // need to be relocalized.
|
| - // 2) An externally installed extension will be found and installed.
|
| - // Note that this can also fail if the app_id is simply invalid.
|
| - // TODO(rafaelw): Do something reasonable here. Pop up a warning panel?
|
| + // This can fail if the app_id is invalid. It can also fail if the
|
| + // extension is external, and has not yet been installed.
|
| + // TODO(skerner): Do something reasonable here. Pop up a warning panel?
|
| // Open an URL to the gallery page of the extension id?
|
| if (!app_id.empty())
|
| return Browser::OpenApplication(profile, app_id, NULL) != NULL;
|
|
|