Move Android to use new Popup Blocker API.

chrome/browser/android/tab_android.h

Index: chrome/browser/android/tab_android.h
diff --git a/chrome/browser/android/tab_android.h b/chrome/browser/android/tab_android.h
index 19d8f25cec0fdc7347a2d715b158df4c485ea60f..21ca338b80c2aff088336da3e3ea570071fb61f6 100644
--- a/chrome/browser/android/tab_android.h
+++ b/chrome/browser/android/tab_android.h
@@ -20,6 +20,10 @@ namespace browser_sync {
 class SyncedTabDelegate;
 }
 
+namespace chrome {
+struct NavigateParams;
+}
+
 namespace content {
 struct ContextMenuParams;
 class WebContents;
@@ -44,6 +48,8 @@ class TabAndroid {
 
   const SessionID& id() const { return tab_id_; }
 
+  virtual void HandleNavigation(chrome::NavigateParams* params) = 0;

[Yaron, 2013/08/22 00:36:30]

I don't think TestShell will link. To this end, could we test popup blocker in
testshell after this change or does it also require
https://codereview.chromium.org/23020009/?

Also, it's a little strange that this is only used by blocked popup case (i.e.
someone looking at the code might expect all navigations to go through here). We
discussed this briefly offline and I think it'd be hard to get all navigations
to go through this one path. Maybe instead we should rename this so it's clear
it's for handling a navigation with a popup.

+
   virtual void OnReceivedHttpAuthRequest(jobject auth_handler,
                                          const string16& host,
                                          const string16& realm) = 0;
@@ -90,11 +96,11 @@ class TabAndroid {
 
   static bool RegisterTabAndroid(JNIEnv* env);
 
+  static void InitTabHelpers(content::WebContents* web_contents);
+
  protected:
   virtual ~TabAndroid();
 
-  static void InitTabHelpers(content::WebContents* web_contents);
-
   content::WebContents* InitWebContentsFromView(JNIEnv* env,
                                                 jobject content_view);