Fix browser tests on Mac

headless/lib/headless_web_contents_browsertest.cc

Index: headless/lib/headless_web_contents_browsertest.cc
diff --git a/headless/lib/headless_web_contents_browsertest.cc b/headless/lib/headless_web_contents_browsertest.cc
index f920a0ccbcff0e6572d66090b8bdc6f65657740f..67fee60f2086f5f0d404b30be5e08fb45c5130e1 100644
--- a/headless/lib/headless_web_contents_browsertest.cc
+++ b/headless/lib/headless_web_contents_browsertest.cc
@@ -88,11 +88,17 @@ IN_PROC_BROWSER_TEST_F(HeadlessWebContentsTest, Focus) {
 
   // TODO(irisu): Focus of two web contents should be independent of the other.
   // Both web_contents and web_contents2 should be focused at this point.
+  // NOTE(dvallet): Mac headless implementation already does this.

[Eric Seckler, 2017/03/28 07:39:48]

Interesting, do you know why that is? Is that simply "normal" on mac/cocoa, or
is it due to something in headless-mac setup?

[dvallet, 2017/03/29 05:16:27]

Yeah I think is a cocoa thing. The window is hidden not shown, but when we do
web_contents_delegate_->ActivateContents(...) then contents are set as focused
regardless of any other inactive windows. I'm happy that this is the case,
otherwise it could've been a problem to set up properly...

   EXPECT_TRUE(EvaluateScript(web_contents, "document.hasFocus()")
                   ->GetResult()
                   ->GetValue()
                   ->GetAsBoolean(&result));
+#if !defined(OS_MACOSX)
   EXPECT_FALSE(result);
+#else
+  // Mac sets all contents as focused.
+  EXPECT_TRUE(result);
+#endif
   EXPECT_TRUE(EvaluateScript(web_contents2, "document.hasFocus()")
                   ->GetResult()
                   ->GetValue()