[Cronet] Test the libcronet that's shipped, not libcronet_test

components/cronet/android/test/cronet_test_util.h

 // Copyright 2016 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_TEST_CRONET_TEST_UTIL_H_
 #define COMPONENTS_CRONET_ANDROID_TEST_CRONET_TEST_UTIL_H_
 
 #include <jni.h>
+#include "base/android/jni_android.h"
+#include "base/memory/ref_counted.h"
+#include "base/single_thread_task_runner.h"
+
+namespace net {
+class URLRequest;
+class URLRequestContext;
+}  // namespace net
 
 namespace cronet {
 
-bool RegisterCronetTestUtil(JNIEnv* env);
+// Various test utility functions for testing Cronet.
+class TestUtil {

[xunjieli, 2016/10/27 14:49:42]

I guess it doesn't make sense for this to have a copy and assignment
constructor.

Could you add a
DISALLOW_COPY_AND_ASSIGN(TestUtil)?

and #include "base/macros.h"

[pauljensen, 2016/10/28 15:24:42]

I've gone one step further and used DISALLOW_IMPLICIT_CONSTRUCTORS().

+ public:
+  // CronetURLRequestContextAdapter manipulation:
+
+  // Returns SingleThreadTaskRunner for the network thread of the context
+  // adapter.
+  static scoped_refptr<base::SingleThreadTaskRunner> GetTaskRunner(
+      jlong jcontext_adapter);
+  // Returns underlying URLRequestContext.
+  static net::URLRequestContext* GetURLRequestContext(jlong jcontext_adapter);
+  // Run |task| after URLRequestContext is initialized.
+  static void RunAfterContextInit(jlong jcontext_adapter,
+                                  const base::Closure& task);
+
+  // CronetURLRequestAdapter manipulation:
+
+  // Returns underlying URLRequest.
+  static net::URLRequest* GetURLRequest(jlong jrequest_adapter);
+
+  // Register JNI.
+  static bool Register(JNIEnv* env);
+
+ private:
+  static void RunAfterContextInitOnNetworkThread(jlong jcontext_adapter,

[xunjieli, 2016/10/27 14:49:42]

The style guide encourages making this file local, see:

http://dev.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Move-st...

[pauljensen, 2016/10/28 15:24:42]

This function needs to be a member of TestUtil because it accesses internals of
CronetURLRequestContext.  I've added a note at the top of this class to clarify.

+                                                 const base::Closure& task);
+};
 
 }  // namespace cronet
 
 #endif  // COMPONENTS_CRONET_ANDROID_TEST_CRONET_TEST_UTIL_H_