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

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 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 <set>
 #include <utility>
 #include <vector>
 
@@ skipping 138 matching lines @@
 #include "chrome/browser/chromeos/fileapi/file_system_backend.h"
 #include "chrome/browser/chromeos/fileapi/mtp_file_system_backend_delegate.h"
 #include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/system/input_device_settings.h"
 #include "chromeos/chromeos_switches.h"
 #include "components/user_manager/user_manager.h"
 #elif defined(OS_LINUX)
 #include "chrome/browser/chrome_browser_main_linux.h"
 #elif defined(OS_ANDROID)
 #include "chrome/browser/android/new_tab_page_url_handler.h"
+#include "chrome/browser/android/service_tab_launcher.h"
 #include "chrome/browser/android/webapps/single_tab_mode_tab_helper.h"
 #include "chrome/browser/chrome_browser_main_android.h"
 #include "chrome/common/descriptors_android.h"
 #include "components/crash/browser/crash_dump_manager_android.h"
 #elif defined(OS_POSIX)
 #include "chrome/browser/chrome_browser_main_posix.h"
 #endif
 
 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
 #include "chrome/browser/media/protected_media_identifier_permission_context.h"
@@ skipping 2333 matching lines @@
 
 bool ChromeContentBrowserClient::CheckMediaAccessPermission(
     content::BrowserContext* browser_context,
     const GURL& security_origin,
     content::MediaStreamType type) {
   return MediaCaptureDevicesDispatcher::GetInstance()
       ->CheckMediaAccessPermission(
           browser_context, security_origin, type);
 }
 
-content::WebContents* ChromeContentBrowserClient::OpenURL(
+void ChromeContentBrowserClient::OpenURL(
     content::BrowserContext* browser_context,
-    const content::OpenURLParams& params) {
+    const content::OpenURLParams& params,
+    base::Callback<void(content::WebContents*)> callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
   NavigateParams nav_params(Profile::FromBrowserContext(browser_context),
                             params.url,
                             params.transition);
   FillNavigateParamsFromOpenURLParams(&nav_params, params);
   nav_params.user_gesture = params.user_gesture;
 
   Navigate(&nav_params);
-  return nav_params.target_contents;
+  callback.Run(nav_params.target_contents);
+#elif defined(OS_ANDROID)
+  chrome::android::ServiceTabLauncher::GetInstance()->LaunchTab(
+      browser_context, params, callback);
 #else
-  // TODO(mlamouri): write a chrome::Navigate() method for Android and iOS.
+  // TODO(mlamouri): write a chrome::Navigate() method for iOS.
   // See https://crbug.com/448409.

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

I think you can remove the TODO and the mention of the BUG. Just change the
comment to note that we do not support this on iOS.

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

Done.

-  return nullptr;
-#endif // !defined(OS_ANDROID) && !defined(OS_IOS)
+  NOTIMPLEMENTED();
+#endif
 }
 
 content::DevToolsManagerDelegate*
 ChromeContentBrowserClient::GetDevToolsManagerDelegate() {
 #if defined(OS_ANDROID)
   return new DevToolsManagerDelegateAndroid();
 #else
   return new ChromeDevToolsManagerDelegate();
 #endif
 }
@@ skipping 68 matching lines @@
       switches::kDisableWebRtcEncryption,
     };
     to_command_line->CopySwitchesFrom(from_command_line,
                                       kWebRtcDevSwitchNames,
                                       arraysize(kWebRtcDevSwitchNames));
   }
 }
 #endif  // defined(ENABLE_WEBRTC)
 
 }  // namespace chrome