[Cronet] Surface SSL cert error through CronetURLRequestAdapter::OnSSLCertificateError

components/cronet/android/cronet_url_request_adapter.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_
 #define COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_
 
 #include <jni.h>
 #include <string>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "net/base/request_priority.h"
 #include "net/url_request/url_request.h"
 #include "url/gurl.h"
 
 namespace base {
 class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace net {
 class HttpRequestHeaders;
 class HttpResponseHeaders;
+class SSLInfo;
 class UploadDataStream;
 }  // namespace net
 
 namespace cronet {
 
 class CronetURLRequestContextAdapter;
 
 bool CronetUrlRequestAdapterRegisterJni(JNIEnv* env);
 
 // An adapter from Java CronetUrlRequest object to net::URLRequest.
@@ skipping 74 matching lines @@
   // Returns true if response is coming from the cache.
   jboolean GetWasCached(JNIEnv* env, jobject jcaller) const;
 
   // net::URLRequest::Delegate implementations:
 
   void OnReceivedRedirect(net::URLRequest* request,
                           const net::RedirectInfo& redirect_info,
                           bool* defer_redirect) override;
   void OnResponseStarted(net::URLRequest* request) override;
   void OnReadCompleted(net::URLRequest* request, int bytes_read) override;
+  void OnSSLCertificateError(net::URLRequest* request,
+                             const net::SSLInfo& ssl_info,
+                             bool fatal) override;

[mmenke, 2015/07/17 21:09:18]

Order here should match the order they're declared in URLRequest::Delegate. 
OnSSLCertificateError should go just before OnResponseStarted (That also roughly
corresponds to the order they're called in, though OnSSLCertificateError can be
before or after redirects)

[xunjieli, 2015/07/20 14:56:20]

Done.

 
  private:
   class IOBufferWithByteBuffer;
 
   void StartOnNetworkThread();
   void GetStatusOnNetworkThread(
       const base::android::ScopedJavaGlobalRef<jobject>& jstatus_listener_ref)
       const;
   void FollowDeferredRedirectOnNetworkThread();
   void ReadDataOnNetworkThread(
@@ skipping 19 matching lines @@
 
   scoped_refptr<IOBufferWithByteBuffer> read_buffer_;
   scoped_ptr<net::URLRequest> url_request_;
 
   DISALLOW_COPY_AND_ASSIGN(CronetURLRequestAdapter);
 };
 
 }  // namespace cronet
 
 #endif  // COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_