Implement Webviewclient.onReceivedSslError

android_webview/native/aw_contents.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_contents.h"
 
 #include <sys/system_properties.h>
 
 #include "android_webview/browser/aw_browser_context.h"
 #include "android_webview/browser/aw_browser_main_parts.h"
@@ skipping 17 matching lines @@
 #include "base/pickle.h"
 #include "base/string16.h"
 #include "base/supports_user_data.h"
 #include "cc/layer.h"
 #include "components/navigation_interception/intercept_navigation_delegate.h"
 #include "content/public/browser/android/content_view_core.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/cert_store.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_process_host.h"
+#include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/ssl_status.h"
 #include "jni/AwContents_jni.h"
 #include "net/base/x509_certificate.h"
 #include "skia/ext/refptr.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkDevice.h"
 #include "third_party/skia/include/core/SkPicture.h"
 #include "ui/gfx/android/java_bitmap.h"
@@ skipping 60 matching lines @@
   AwContents* contents_;
 };
 
 }  // namespace
 
 // static
 AwContents* AwContents::FromWebContents(WebContents* web_contents) {
   return AwContentsUserData::GetContents(web_contents);
 }
 
+// static
+// The implementation is here to provide isolation of the browser/ layer
+// from the implementation details of native/ layer.
+AwCertificateErrorHandlerBase* AwCertificateErrorHandlerBase::FromID(
+    int render_process_id,
+    int render_view_id) {
+
+  const content::RenderViewHost* host =

[mkosiba (inactive), 2013/02/04 12:51:31]

uber nit: normally this is called the rvh, not host :)

[sgurun-gerrit only, 2013/02/21 23:32:32]

Done.

+      content::RenderViewHost::FromID(render_process_id, render_view_id);
+  DCHECK(host);

[mkosiba (inactive), 2013/02/04 12:51:31]

please make this robust to NULLs. It's likely for an rvh to not exist in
unittest code (in production it's rarer but still can happen if the user closes
a tab at the "right" time).

[sgurun-gerrit only, 2013/02/21 23:32:32]

Done.

+  content::WebContents* web_contents =
+      content::WebContents::FromRenderViewHost(host);
+  DCHECK(web_contents);
+  return static_cast<AwCertificateErrorHandlerBase*>

[mkosiba (inactive), 2013/02/04 12:51:31]

nit: I think the common pattern is to leave the open paren on the previous line.

[sgurun-gerrit only, 2013/02/21 23:32:32]

Done.

+      (AwContents::FromWebContents(web_contents));
+}
+
 AwContents::AwContents(JNIEnv* env,
                        jobject obj,
                        jobject web_contents_delegate)
     : java_ref_(env, obj),
       web_contents_delegate_(
           new AwWebContentsDelegate(env, web_contents_delegate)),
       view_visible_(false),
       compositor_visible_(false),
       is_composite_pending_(false),
       last_frame_context_(NULL) {
@@ skipping 782 matching lines @@
 void AwContents::OnPictureUpdated(int process_id, int render_view_id) {
   CHECK_EQ(web_contents_->GetRenderProcessHost()->GetID(), process_id);
   if (render_view_id != web_contents_->GetRoutingID())
     return;
 
   // TODO(leandrogracia): delete when sw rendering uses Ubercompositor.
   // Invalidation should be provided by the compositor only.
   Invalidate();
 }
 
+void AwContents::AllowCertificateError(
+    int cert_error,
+    net::X509Certificate* cert,
+    const GURL& request_url,
+    const base::Callback<void(bool)>& callback,
+    bool* cancel_request) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  JNIEnv* env = AttachCurrentThread();
+
+  std::string der_string;
+  net::X509Certificate::GetDEREncoded(cert->os_cert_handle(),&der_string);
+  ScopedJavaLocalRef<jbyteArray> jcert = base::android::ToJavaByteArray(env,
+      reinterpret_cast<const uint8*>(der_string.data()), der_string.length());
+  ScopedJavaLocalRef<jstring> jurl(ConvertUTF8ToJavaString(
+      env, request_url.spec()));
+  ssl_error_callback_ = callback;

[boliu, 2013/02/02 03:51:29]

Only save callback if cancel_request is false? But I guess you are already
reworking this part of the code.

[mkosiba (inactive), 2013/02/04 12:51:31]

IMO it would be helpful if we had a bridge from Chromium base::Callbacks to a
Java object (something like Callable<?> maybe). We've been working around the
lack of such a construct on multiple occasions (this one included) so I think
it'd be a good investment. WDYT?

[sgurun-gerrit only, 2013/02/21 23:32:32]

Makes sense.  adding a bridge for the aw_content_client as we talked. It seems
you are working on the native callback bridge.

On 2013/02/04 12:51:31, Martin Kosiba wrote:

[sgurun-gerrit only, 2013/02/21 23:32:32]

we are now saving all the callbacks.

On 2013/02/02 03:51:29, boliu wrote:

+  *cancel_request = Java_AwContents_cancelCertificateError(env,
+      java_ref_.get(env).obj(), reinterpret_cast<jint>(cert_error), jcert.obj(),
+      jurl.obj());
+}
+
+void AwContents::ProceedSslError(JNIEnv* env, jobject obj, jboolean proceed) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  if (ssl_error_callback_.is_null()) {
+    LOG(WARNING) << "Ignoring unexpected ssl error proceed callback";
+    return;
+  }
+  ssl_error_callback_.Run(proceed);
+  ssl_error_callback_.Reset();
+}
+
+
 }  // namespace android_webview