Add "open as window" menu item to NTP app menu.

chrome/browser/extensions/extension_dom_ui.cc

Index: chrome/browser/extensions/extension_dom_ui.cc
diff --git a/chrome/browser/extensions/extension_dom_ui.cc b/chrome/browser/extensions/extension_dom_ui.cc
index 6f16484cedb7864b991bc0b88e416a22e53ca90f..76d29acee317dbe1af1f5d09a14831fb5cc4af70 100644
--- a/chrome/browser/extensions/extension_dom_ui.cc
+++ b/chrome/browser/extensions/extension_dom_ui.cc
@@ -239,9 +239,6 @@ bool ExtensionDOMUI::HandleChromeURLOverride(GURL* url, Profile* profile) {
 
   ExtensionsService* service = profile->GetExtensionsService();
   if (!service->is_ready()) {
-    // TODO(erikkay) So far, it looks like extensions load before the new tab
-    // page.  I don't know if we have anything that enforces this, so add this
-    // check for safety.
     NOTREACHED() << "Chrome URL override requested before extensions loaded";

[Sam Kerner (Chrome), 2010/11/17 05:07:36]

Comment is obsolete now that extensions service loads synchronously.

[Aaron Boodman, 2010/11/17 06:12:47]

In that case, you can remove this check completely. Also, maybe we should remove
is_ready() since ExtensionsService is always ready now?

[Sam Kerner (Chrome), 2010/11/18 03:53:36]

ExtensionsService::ready_ is set to false in the extensions service constructor,
and set to true by a call to ExtensionsService::Init().  Kept the internal bool
as a way to catch multiple Init() calls.  Got rid of is_ready().

     return false;
   }