[Android WebView] AwContentsClient.shouldCreateWindow callback part 1.

android_webview/native/aw_web_contents_delegate.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 "android_webview/native/aw_web_contents_delegate.h"
 
 #include "base/lazy_instance.h"
 #include "android_webview/browser/find_helper.h"
 #include "android_webview/native/aw_contents.h"
 #include "android_webview/native/aw_javascript_dialog_creator.h"
 #include "content/public/browser/android/download_controller_android.h"
 #include "content/public/browser/web_contents.h"
+#include "jni/AwWebContentsDelegate_jni.h"
 #include "net/http/http_request_headers.h"
 
+using base::android::AttachCurrentThread;
 using content::WebContents;
 
 namespace android_webview {
 
 static base::LazyInstance<AwJavaScriptDialogCreator>::Leaky
     g_javascript_dialog_creator = LAZY_INSTANCE_INITIALIZER;
 
 AwWebContentsDelegate::AwWebContentsDelegate(
     JNIEnv* env,
     jobject obj)
@@ skipping 32 matching lines @@
         source, request_id);
   }
   return false;
 }
 
 void AwWebContentsDelegate::OnStartDownload(WebContents* source,
                                             content::DownloadItem* download) {
   NOTREACHED();  // We always return false in CanDownload.
 }
 
+void AwWebContentsDelegate::AddNewContents(content::WebContents* source,

[joth, 2012/11/14 18:34:52]

this method is spurious: WebContentsDelegateAndroid::AddNewContents already does
this, so we just need to override the method java side.

+                              content::WebContents* new_contents,

[joth, 2012/11/14 18:34:52]

wonky indent

[benm (inactive), 2012/11/14 18:42:34]

Done.

+                              WindowOpenDisposition disposition,
+                              const gfx::Rect& initial_pos,
+                              bool user_gesture,
+                              bool* was_blocked) {
+  DCHECK(was_blocked);
+  JNIEnv* env = AttachCurrentThread();
+
+  // TODO(benm): Need to plumb whether this popup window is a "dialog", for
+  // the Classic Android WebViews definition of "dialog".
+  bool is_dialog = false;

[joth, 2012/11/14 18:34:52]

I'd just say is_dialog = (disposition == NEW_POPUP)

This definition seems near enough:-

WebNavigationPolicy ChromeClientImpl::getNavigationPolicy()
{
    // If our default configuration was modified by a script or wasn't
    // created by a user gesture, then show as a popup. Else, let this
    // new window be opened as a toplevel window.
    bool asPopup = !m_toolbarsVisible
        || !m_statusbarVisible
        || !m_scrollbarsVisible
        || !m_menubarVisible
        || !m_resizable;

[benm (inactive), 2012/11/14 18:42:34]

Done.

+  bool create_popup = Java_AwWebContentsDelegate_AddNewContents(env,
+      GetJavaDelegate(env).obj(), is_dialog, user_gesture);
+
+  if (create_popup) {
+    // TODO(benm): Implement :)
+    NOTIMPLEMENTED() << "Popup windows are currently not supported for"
+                     << "the chromium powered Android WebView.";
+    *was_blocked = true;
+  } else {
+    *was_blocked = true;
+  }
+}
+
+bool RegisterAwWebContentsDelegate(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}
+
 }  // namespace android_webview