Use @NativeClassQualifiedName in CronetUrlRequestContext.java

components/cronet/android/cronet_url_request_context_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_CONTEXT_ADAPTER_H_
 #define COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_CONTEXT_ADAPTER_H_
 
+#include <jni.h>

[mmenke, 2015/02/17 16:30:17]

nit:  Blank line between C and C++ headers.

[xunjieli, 2015/02/17 18:29:01]

Done.

 #include <queue>
 #include <string>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread.h"
-#include "net/base/net_log.h"
-#include "net/base/network_change_notifier.h"
 
 namespace base {
 class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace net {
 class NetLogLogger;
 class URLRequestContext;
 class ProxyConfigService;
 }  // namespace net
 
 namespace cronet {
 
+bool CronetUrlRequestContextAdapterRegisterJni(JNIEnv* env);

[mef, 2015/02/19 14:45:07]

nit: I'm not sure, but I think this should go below forward declaration of
struct URLRequestContextConfig;

[xunjieli, 2015/02/19 15:19:33]

Done. Didn't realize struct URLRequestContextConfig was a forward declaration :)

+
 struct URLRequestContextConfig;
 
 // Adapter between Java CronetUrlRequestContext and net::URLRequestContext.
 class CronetURLRequestContextAdapter {
  public:
   explicit CronetURLRequestContextAdapter(
       scoped_ptr<URLRequestContextConfig> context_config);
 
   ~CronetURLRequestContextAdapter();
 
   // Called on main Java thread to initialize URLRequestContext.
-  void InitRequestContextOnMainThread(
-      const base::Closure& java_init_network_thread);
+  void InitRequestContextOnMainThread(JNIEnv* env, jobject jcaller);
 
   // Releases all resources for the request context and deletes the object.
   // Blocks until network thread is destroyed after running all pending tasks.
-  void Destroy();
+  void Destroy(JNIEnv* env, jobject jcaller);
 
   // Posts a task that might depend on the context being initialized
   // to the network thread.
   void PostTaskToNetworkThread(const tracked_objects::Location& posted_from,
                                const base::Closure& callback);
 
   bool IsOnNetworkThread() const;
 
   net::URLRequestContext* GetURLRequestContext();
 
-  void StartNetLogToFile(const std::string& file_name);
+  void StartNetLogToFile(JNIEnv* env, jobject jcaller, jstring jfile_name);
 
-  void StopNetLog();
+  void StopNetLog(JNIEnv* env, jobject jcaller);
 
   // Default net::LOAD flags used to create requests.
   int default_load_flags() const { return default_load_flags_; }
 
   // Called on main Java thread to initialize URLRequestContext.
   void InitRequestContextOnMainThread();
 
  private:
   // Initializes |context_| on the Network thread.
   void InitializeOnNetworkThread(
@@ skipping 25 matching lines @@
   std::queue<base::Closure> tasks_waiting_for_context_;
   bool is_context_initialized_;
   int default_load_flags_;
 
   DISALLOW_COPY_AND_ASSIGN(CronetURLRequestContextAdapter);
 };
 
 }  // namespace cronet
 
 #endif  // COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_CONTEXT_ADAPTER_H_