AppListViewDelegate: Moved TODO to the correct line.

chrome/browser/ui/app_list/app_list_view_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/app_list/app_list_view_delegate.h"
 
 #include <vector>
 
 #include "apps/custom_launcher_page_contents.h"
 #include "base/callback.h"
@@ skipping 103 matching lines @@
   // First, check the command line.
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
   if (app_list::switches::IsExperimentalAppListEnabled() &&
       command_line->HasSwitch(switches::kCustomLauncherPage)) {
     GURL custom_launcher_page_url(
         command_line->GetSwitchValueASCII(switches::kCustomLauncherPage));
 
     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();
     }
   }
 
   // Search the list of installed extensions for ones with 'launcher_page'.
   extensions::ExtensionRegistry* extension_registry =
       extensions::ExtensionRegistry::Get(browser_context);
   const extensions::ExtensionSet& enabled_extensions =
       extension_registry->enabled_extensions();
   for (extensions::ExtensionSet::const_iterator it = enabled_extensions.begin();
        it != enabled_extensions.end();
        ++it) {
     const extensions::Extension* extension = it->get();
     const extensions::Manifest* manifest = extension->manifest();
     if (!manifest->HasKey(extensions::manifest_keys::kLauncherPage))
       continue;
     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.";
       continue;
     }
     urls->push_back(extension->GetResourceURL(launcher_page_page));
   }
 }
 
 }  // namespace
@@ skipping 426 matching lines @@
 
 void AppListViewDelegate::AddObserver(
     app_list::AppListViewDelegateObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void AppListViewDelegate::RemoveObserver(
     app_list::AppListViewDelegateObserver* observer) {
   observers_.RemoveObserver(observer);
 }