AppShell support for Mac.

extensions/shell/browser/shell_browsertest.cc

Index: extensions/shell/browser/shell_browsertest.cc
diff --git a/extensions/shell/browser/shell_browsertest.cc b/extensions/shell/browser/shell_browsertest.cc
index 407e417f0de1cdc1424662a1453e16b8f8e7df51..5e1a86984ea4ac4e04780311c41f10ca443c762f 100644
--- a/extensions/shell/browser/shell_browsertest.cc
+++ b/extensions/shell/browser/shell_browsertest.cc
@@ -10,7 +10,10 @@
 #include "extensions/browser/app_window/app_window_registry.h"
 #include "extensions/browser/notification_types.h"
 #include "extensions/shell/test/shell_apitest.h"
+
+#if defined(USE_AURA)
 #include "ui/aura/window.h"
+#endif
 
 namespace extensions {
 
@@ -23,8 +26,10 @@ IN_PROC_BROWSER_TEST_F(ShellApiTest, Basic) {
       AppWindowRegistry::Get(browser_context())->app_windows().front();
   ASSERT_TRUE(app_window);
 
+#if defined(USE_AURA)
   // The web contents have focus.
   EXPECT_TRUE(app_window->web_contents()->GetContentNativeView()->HasFocus());
+#endif

[James Cook, 2014/12/10 21:05:01]

Is there a way to test for this focus on non-Aura?  Getting this right on Aura
was a pain.

[Yoyo Zhou, 2014/12/11 02:40:50]

I'd like to. Adding a TODO for now. Right now we don't get OS-level focus, only
focus among our application's windows; see the TODO in
ShellNativeAppWindowMac::Activate. There's a utility function in
chrome/browser/ui/cocoa/browser_window_utils.h that does this; obviously, we'd
need to refactor it out of chrome/ to use it.

 }
 
 }  // namespace extensions