Remove unnecessary |result| argument from AllowCertificateError()

android_webview/browser/aw_content_browser_client.cc

Index: android_webview/browser/aw_content_browser_client.cc
diff --git a/android_webview/browser/aw_content_browser_client.cc b/android_webview/browser/aw_content_browser_client.cc
index 942b0c368ec4d4d8a51030f7c410db916ab7e7d6..5209fad3f5b9b214f18ca3f6ebe3aa777c148910 100644
--- a/android_webview/browser/aw_content_browser_client.cc
+++ b/android_webview/browser/aw_content_browser_client.cc
@@ -352,8 +352,8 @@ void AwContentBrowserClient::AllowCertificateError(
     bool overridable,
     bool strict_enforcement,
     bool expired_previous_decision,
-    const base::Callback<void(bool)>& callback,
-    content::CertificateRequestResultType* result) {
+    const base::Callback<void(content::CertificateRequestResultType)>&
+        callback) {
   AwContentsClientBridgeBase* client =
       AwContentsClientBridgeBase::FromWebContents(web_contents);
   bool cancel_request = true;
@@ -363,8 +363,9 @@ void AwContentBrowserClient::AllowCertificateError(
                                   request_url,
                                   callback,
                                   &cancel_request);
-  if (cancel_request)
-    *result = content::CERTIFICATE_REQUEST_RESULT_TYPE_DENY;
+  if (cancel_request) {

[nasko, 2016/08/08 18:32:03]

nit: No need for {} on one-line if statements.

[estark, 2016/08/08 20:00:13]

Done.

+    callback.Run(content::CERTIFICATE_REQUEST_RESULT_TYPE_DENY);
+  }
 }
 
 void AwContentBrowserClient::SelectClientCertificate(