Add ShellAppsClient, ShellAppDelegate, ShellNativeAppWindow

extensions/shell/browser/desktop_controller.h

 // Copyright 2014 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.
 
 #ifndef EXTENSIONS_SHELL_BROWSER_DESKTOP_CONTROLLER_H_
 #define EXTENSIONS_SHELL_BROWSER_DESKTOP_CONTROLLER_H_
 
 namespace aura {
+class Window;
 class WindowTreeHost;
 }
 
 namespace content {
 class BrowserContext;
 }
 
 namespace extensions {
 class Extension;
 class ShellAppWindow;
@@ skipping 14 matching lines @@
 
   // Returns the WindowTreeHost created by this DesktopController.
   virtual aura::WindowTreeHost* GetHost() = 0;
 
   // Creates a new app window and adds it to the desktop. The desktop maintains
   // ownership of the window. The window must be closed before |extension| is
   // destroyed.
   virtual ShellAppWindow* CreateAppWindow(content::BrowserContext* context,
                                           const Extension* extension) = 0;
 
+  // Attaches the window to our window hierarchy.
+  virtual void AddAppWindow(aura::Window* window) = 0;

[oshima, 2014/09/05 18:06:08]

What I had in my mind was that CreateAppWindow will be gone and replaced with
AppsClient, and athena_main can use it's own impl, instead of using
DesktopController. 

Is there an issue with this approach?

+
   // Closes and destroys the app windows.
   virtual void CloseAppWindows() = 0;
 };
 
 }  // namespace extensinos
 
 #endif  // EXTENSIONS_SHELL_BROWSER_DESKTOP_CONTROLLER_H_