Add support for chrome.app.window.create() to app_shell

apps/shell/browser/shell_browser_main_parts.cc

Index: apps/shell/browser/shell_browser_main_parts.cc
diff --git a/apps/shell/browser/shell_browser_main_parts.cc b/apps/shell/browser/shell_browser_main_parts.cc
index 91594811992b71e4da72790193f230238fe1deb4..dae80bb2ffb9e4b3c4bf3586430b3c4ec078aacb 100644
--- a/apps/shell/browser/shell_browser_main_parts.cc
+++ b/apps/shell/browser/shell_browser_main_parts.cc
@@ -4,6 +4,7 @@
 
 #include "apps/shell/browser/shell_browser_main_parts.h"
 
+#include "apps/shell/browser/shell_apps_client.h"
 #include "apps/shell/browser/shell_browser_context.h"
 #include "apps/shell/browser/shell_extension_system.h"
 #include "apps/shell/browser/shell_extension_system_factory.h"
@@ -111,6 +112,9 @@ void ShellBrowserMainParts::PreMainMessageLoopRun() {
       new extensions::ShellExtensionsBrowserClient(browser_context_.get()));
   extensions::ExtensionsBrowserClient::Set(extensions_browser_client_.get());
 
+  apps_client_.reset(new ShellAppsClient(browser_context_.get()));
+  AppsClient::Set(apps_client_.get());
+
   // Create our custom ExtensionSystem first because other
   // BrowserContextKeyedServices depend on it.
   // TODO(yoz): Move this after EnsureBrowserContextKeyedServiceFactoriesBuilt.
@@ -163,6 +167,7 @@ void ShellBrowserMainParts::PostMainMessageLoopRun() {
 
 void ShellBrowserMainParts::OnWindowTreeHostCloseRequested(

[Haojian Wu, 2014/02/25 13:48:48]

James@, could you tell me when will this method get invoked?

I have done some tracks of this method, I don't see any invokes of this method.
 
From the
document:https://code.google.com/p/chromium/codesearch#chromium/src/ui/aura/root_window_observer.h&q=nWindowTreeHostCloseRequested&sq=package:chromium&type=cs&l=31
, it says "Invoked when the native windowing system sends us a request to close
our window.". Does this mean when we close the button of app_shell root window,
it will be invoked?

[James Cook, 2014/02/25 17:47:32]

I'm not very familiar with that method, but it sounds like it should be called
when the user clicks the close button of the root window.  Someone refactored
this recently -- you might look at the svn blame / git blame for this line and
see what they changed.  Maybe we're not observing the root window correctly.

     const aura::WindowEventDispatcher* dispatcher) {
+  extension_system_->CloseApp();
   base::MessageLoop::current()->PostTask(FROM_HERE,
                                          base::MessageLoop::QuitClosure());
 }