Initial implementation of Cronet Async API.

components/cronet/android/cronet_url_request_context.cc

Index: components/cronet/android/cronet_url_request_context.cc
diff --git a/components/cronet/android/chromium_url_request_context.cc b/components/cronet/android/cronet_url_request_context.cc
similarity index 72%
copy from components/cronet/android/chromium_url_request_context.cc
copy to components/cronet/android/cronet_url_request_context.cc
index 9ef8c8860f4ae7c0a06b619a108ef88bad57f379..a208646671e2866a67f0ae97c6ec3bcf33674616 100644
--- a/components/cronet/android/chromium_url_request_context.cc
+++ b/components/cronet/android/cronet_url_request_context.cc
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "components/cronet/android/chromium_url_request_context.h"
+#include "components/cronet/android/cronet_url_request_context.h"
 
 #include <string>
 
@@ -13,33 +13,34 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/statistics_recorder.h"
 #include "base/values.h"
-#include "components/cronet/android/chromium_url_request.h"
-#include "components/cronet/android/url_request_adapter.h"
-#include "components/cronet/android/url_request_context_adapter.h"
+#include "components/cronet/android/cronet_url_request.h"
+#include "components/cronet/android/cronet_url_request_adapter.h"
+#include "components/cronet/android/cronet_url_request_context_adapter.h"
 #include "components/cronet/url_request_context_config.h"
-#include "jni/ChromiumUrlRequestContext_jni.h"
+#include "jni/CronetUrlRequestContext_jni.h"
 
 namespace {
 
-// Delegate of URLRequestContextAdapter that delivers callbacks to the Java
+// Delegate of CronetURLRequestContextAdapter that delivers callbacks to the
+// Java
 // layer.
-class JniURLRequestContextAdapterDelegate
-    : public cronet::URLRequestContextAdapter::
-          URLRequestContextAdapterDelegate {
+class JniCronetURLRequestContextAdapterDelegate
+    : public cronet::CronetURLRequestContextAdapter::
+          CronetURLRequestContextAdapterDelegate {
  public:
-  JniURLRequestContextAdapterDelegate(JNIEnv* env, jobject owner)
+  JniCronetURLRequestContextAdapterDelegate(JNIEnv* env, jobject owner)
       : owner_(env->NewGlobalRef(owner)) {}
 
-  virtual void OnContextInitialized(
-      cronet::URLRequestContextAdapter* context) override {
+  void OnContextInitialized(
+      cronet::CronetURLRequestContextAdapter* context) override {
     JNIEnv* env = base::android::AttachCurrentThread();
-    cronet::Java_ChromiumUrlRequestContext_initNetworkThread(env, owner_);
+    cronet::Java_CronetUrlRequestContext_initNetworkThread(env, owner_);
     // TODO(dplotnikov): figure out if we need to detach from the thread.
     // The documentation says we should detach just before the thread exits.
   }
 
  protected:
-  virtual ~JniURLRequestContextAdapterDelegate() {
+  virtual ~JniCronetURLRequestContextAdapterDelegate() {
     JNIEnv* env = base::android::AttachCurrentThread();
     env->DeleteGlobalRef(owner_);
   }
@@ -53,7 +54,7 @@ class JniURLRequestContextAdapterDelegate
 namespace cronet {
 
 // Explicitly register static JNI functions.
-bool ChromiumUrlRequestContextRegisterJni(JNIEnv* env) {
+bool CronetUrlRequestContextRegisterJni(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
@@ -61,12 +62,8 @@ bool ChromiumUrlRequestContextRegisterJni(JNIEnv* env) {
 static jlong CreateRequestContextAdapter(JNIEnv* env,
                                          jobject object,
                                          jobject context,
-                                         jstring user_agent,
                                          jint log_level,
                                          jstring config) {
-  std::string user_agent_string =
-      base::android::ConvertJavaStringToUTF8(env, user_agent);
-
   std::string config_string =
       base::android::ConvertJavaStringToUTF8(env, config);
 
@@ -93,8 +90,8 @@ static jlong CreateRequestContextAdapter(JNIEnv* env,
   logging::SetMinLogLevel(static_cast<int>(log_level));
 
   // TODO(dplotnikov): set application context.
-  URLRequestContextAdapter* adapter = new URLRequestContextAdapter(
-      new JniURLRequestContextAdapterDelegate(env, object), user_agent_string);
+  CronetURLRequestContextAdapter* adapter = new CronetURLRequestContextAdapter(

[xunjieli, 2014/10/17 00:31:34]

nit: There are two types of adapter, one is for the request, the other one is
for the context. Can we rename |adapter| to |context_adapter| to make it
clearer?

[mef, 2014/10/17 20:19:42]

Done.

+      new JniCronetURLRequestContextAdapterDelegate(env, object));
   adapter->AddRef();  // Hold onto this ref-counted object.
   adapter->Initialize(context_config.Pass());
   return reinterpret_cast<jlong>(adapter);
@@ -104,11 +101,13 @@ static jlong CreateRequestContextAdapter(JNIEnv* env,
 static void ReleaseRequestContextAdapter(JNIEnv* env,
                                          jobject object,
                                          jlong urlRequestContextAdapter) {
-  URLRequestContextAdapter* adapter =
-      reinterpret_cast<URLRequestContextAdapter*>(urlRequestContextAdapter);
+  CronetURLRequestContextAdapter* adapter =
+      reinterpret_cast<CronetURLRequestContextAdapter*>(
+          urlRequestContextAdapter);
   // TODO(mef): Revisit this from thread safety point of view: Can we delete a
   // thread while running on that thread?
-  // URLRequestContextAdapter is a ref-counted object, and may have pending
+  // CronetURLRequestContextAdapter is a ref-counted object, and may have
+  // pending
   // tasks,

[xunjieli, 2014/10/17 00:31:34]

nit: formatting.

[mef, 2014/10/17 20:19:42]

Done.

   // so we need to release it instead of deleting here.
   adapter->Release();
@@ -132,8 +131,9 @@ static void StartNetLogToFile(JNIEnv* env,
                               jobject jcaller,
                               jlong urlRequestContextAdapter,

[xunjieli, 2014/10/17 00:31:35]

nit: c_style_naming with a "j" prefix. eg. jurl_request_context_adapter

[mef, 2014/10/17 20:19:42]

Done.

                               jstring fileName) {
-  URLRequestContextAdapter* adapter =
-      reinterpret_cast<URLRequestContextAdapter*>(urlRequestContextAdapter);
+  CronetURLRequestContextAdapter* adapter =
+      reinterpret_cast<CronetURLRequestContextAdapter*>(
+          urlRequestContextAdapter);
   std::string file_name = base::android::ConvertJavaStringToUTF8(env, fileName);
   adapter->StartNetLogToFile(file_name);
 }
@@ -142,8 +142,9 @@ static void StartNetLogToFile(JNIEnv* env,
 static void StopNetLog(JNIEnv* env,
                        jobject jcaller,
                        jlong urlRequestContextAdapter) {
-  URLRequestContextAdapter* adapter =
-      reinterpret_cast<URLRequestContextAdapter*>(urlRequestContextAdapter);
+  CronetURLRequestContextAdapter* adapter =
+      reinterpret_cast<CronetURLRequestContextAdapter*>(
+          urlRequestContextAdapter);
   adapter->StopNetLog();
 }