Index: chrome/browser/ui/app_list/app_list_view_delegate.cc |
diff --git a/chrome/browser/ui/app_list/app_list_view_delegate.cc b/chrome/browser/ui/app_list/app_list_view_delegate.cc |
index be225dea64b5c34119d9d4d0822c48a950b9a963..f84222792063e2c41d279023bb0b2f8d0a30d058 100644 |
--- a/chrome/browser/ui/app_list/app_list_view_delegate.cc |
+++ b/chrome/browser/ui/app_list/app_list_view_delegate.cc |
@@ -121,8 +121,6 @@ void GetCustomLauncherPageUrls(content::BrowserContext* browser_context, |
if (custom_launcher_page_url.SchemeIs(extensions::kExtensionScheme)) { |
urls->push_back(custom_launcher_page_url); |
} else { |
- // TODO(mgiuca): Add a proper manifest parser to catch this error properly |
- // and display it on the extensions page. |
LOG(ERROR) << "Invalid custom launcher page URL: " |
<< custom_launcher_page_url.possibly_invalid_spec(); |
} |
@@ -143,6 +141,8 @@ void GetCustomLauncherPageUrls(content::BrowserContext* browser_context, |
std::string launcher_page_page; |
if (!manifest->GetString(extensions::manifest_keys::kLauncherPagePage, |
&launcher_page_page)) { |
+ // TODO(mgiuca): Add a proper manifest parser to catch this error properly |
+ // and display it on the extensions page. |
LOG(ERROR) << "Extension " << extension->id() << ": " |
<< extensions::manifest_keys::kLauncherPage |
<< " has no 'page' attribute; will be ignored."; |