Revert 66646 - Add "open as window" menu item to NTP app menu....

chrome/browser/ui/browser_init.cc

 // Copyright (c) 2010 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/browser_init.h"
 
 #include <algorithm>   // For max().
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
@@ skipping 587 matching lines @@
     return true;
   }
   return false;
 }
 
 bool BrowserInit::LaunchWithProfile::OpenApplicationWindow(Profile* profile) {
   std::string url_string, app_id;
   if (!IsAppLaunch(&url_string, &app_id))
     return false;
 
-  // 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?
+  // 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?
   // Open an URL to the gallery page of the extension id?
   if (!app_id.empty())
     return Browser::OpenApplication(profile, app_id, NULL) != NULL;
 
   if (url_string.empty())
     return false;
 
 #if defined(OS_WIN)  // Fix up Windows shortcuts.
   ReplaceSubstringsAfterOffset(&url_string, 0, "\\x", "%");
 #endif
@@ skipping 401 matching lines @@
   scoped_refptr<AutomationProviderClass> automation =
       new AutomationProviderClass(profile);
   automation->ConnectToChannel(channel_id);
   automation->SetExpectedTabCount(expected_tabs);
 
   AutomationProviderList* list =
       g_browser_process->InitAutomationProviderList();
   DCHECK(list);
   list->AddProvider(automation);
 }