First step in enabling creating tabs without an Activity on Android.

chrome/browser/chrome_content_browser_client_unittest.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/chrome_content_browser_client.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 14 matching lines @@
 
 using ChromeContentBrowserClientTest = testing::Test;
 
 TEST_F(ChromeContentBrowserClientTest, ShouldAssignSiteForURL) {
   ChromeContentBrowserClient client;
   EXPECT_FALSE(client.ShouldAssignSiteForURL(GURL("chrome-native://test")));
   EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("http://www.google.com")));
   EXPECT_TRUE(client.ShouldAssignSiteForURL(GURL("https://www.google.com")));
 }
 
-using ChromeContentBrowserClientWindowTest = BrowserWithTestWindowTest;
-
 // BrowserWithTestWindowTest doesn't work on iOS and Android.
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
 
+class ChromeContentBrowserClientWindowTest : public BrowserWithTestWindowTest {
+ public:
+  ChromeContentBrowserClientWindowTest()
+      : web_contents_(nullptr) {}
+
+  void DidOpenURL(content::WebContents* web_contents) {
+    web_contents_ = web_contents;
+  }
+
+ protected:
+  content::WebContents* web_contents() const { return web_contents_; }
+
+ private:
+  content::WebContents* web_contents_;
+};
+
 // This test opens two URLs using ContentBrowserClient::OpenURL. It expects the
 // URLs to be opened in new tabs and activated, changing the active tabs after
 // each call and increasing the tab count by 2.
 TEST_F(ChromeContentBrowserClientWindowTest, OpenURL) {
   ChromeContentBrowserClient client;
 
   int previous_count = browser()->tab_strip_model()->count();
 
   GURL urls[] = { GURL("https://www.google.com"),
                   GURL("https://www.chromium.org") };
 
   for (const GURL& url : urls) {
     content::OpenURLParams params(url,
                                   content::Referrer(),
                                   NEW_FOREGROUND_TAB,
                                   ui::PAGE_TRANSITION_AUTO_TOPLEVEL,
                                   false);
-    content::WebContents* contents =
-        client.OpenURL(browser()->profile(), params);
-    EXPECT_TRUE(contents);
+    // TODO(peter): We should have more in-depth browser tests for the window
+    // opening functionality, which also covers Android. For all platforms but
+    // Android and iOS this is synchronous, which is the assumption this test

[mlamouri (slow - plz ping), 2015/02/09 12:07:37]

nit: I wouldn't make any assumption about iOS implementation in the comments.

[Peter Beverloo, 2015/02/10 17:49:37]

Done.

+    // currently follows.
+    client.OpenURL(browser()->profile(),
+                   params,
+                   base::Bind(&ChromeContentBrowserClientWindowTest::DidOpenURL,
+                              base::Unretained(this)));
+    EXPECT_TRUE(web_contents());

[mlamouri (slow - plz ping), 2015/02/09 12:07:37]

It's a bit odd to have web_contents() being a test method even though it doesn't
mean anything for the test itself. Probably not a big deal but I wonder why you
preferred that compared to have a DidOpenURL anonymous method that would take a
WebContents** and set it based on the callback return value. It would also make
the synchronous requirement way more obvious because you would set the
WebContents* on the stack, dereference it to send it to the callback, then read
the value again on the stack.

WDYT? Feel free to keep the current design if you prefer it.

[Peter Beverloo, 2015/02/10 17:49:37]

Done.

 
     content::WebContents* active_contents = browser()->tab_strip_model()->
         GetActiveWebContents();
-    EXPECT_EQ(contents, active_contents);
+    EXPECT_EQ(web_contents(), active_contents);
     EXPECT_EQ(url, active_contents->GetVisibleURL());
   }
 
   EXPECT_EQ(previous_count + 2, browser()->tab_strip_model()->count());
 }
 
 #endif // !defined(OS_ANDROID) && !defined(OS_IOS)
 
 #if defined(ENABLE_WEBRTC)
 
@@ skipping 162 matching lines @@
 
 #if defined(OS_ANDROID)
   SetPermission(CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER,
                 CONTENT_SETTING_ALLOW);
   CheckPermissionStatus(PERMISSION_PROTECTED_MEDIA_IDENTIFIER,
                         PERMISSION_STATUS_GRANTED);
 #endif
 }
 
 }  // namespace chrome