Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(390)

Unified Diff: components/cronet/android/test/sdch_test_util.cc

Issue 2406273002: [Cronet] Test the libcronet that's shipped, not libcronet_test (Closed)
Patch Set: fix perf test Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/cronet/android/test/sdch_test_util.cc
diff --git a/components/cronet/android/test/sdch_test_util.cc b/components/cronet/android/test/sdch_test_util.cc
index 8d74ea287ddea89dbc7303253a0e5dc8c3c6fbd0..1c93d018587e6981d5ec154b3ca95bb063a021bd 100644
--- a/components/cronet/android/test/sdch_test_util.cc
+++ b/components/cronet/android/test/sdch_test_util.cc
@@ -11,7 +11,6 @@
#include "base/android/scoped_java_ref.h"
#include "base/bind.h"
#include "base/macros.h"
-#include "components/cronet/android/cronet_url_request_context_adapter.h"
#include "jni/SdchObserver_jni.h"
#include "net/base/sdch_manager.h"
#include "net/base/sdch_observer.h"
@@ -80,20 +79,12 @@ void AddSdchObserverHelper(
Java_SdchObserver_onAddSdchObserverCompleted(env, jsdch_observer_ref);
}
-void AddSdchObserverOnNetworkThread(
- const GURL& target_url,
- const base::android::ScopedJavaGlobalRef<jobject>& jsdch_observer_ref,
- CronetURLRequestContextAdapter* context_adapter) {
- AddSdchObserverHelper(target_url, jsdch_observer_ref,
- context_adapter->GetURLRequestContext());
-}
-
} // namespace
-void AddSdchObserver(JNIEnv* env,
- const JavaParamRef<jobject>& jsdch_observer,
- const JavaParamRef<jstring>& jtarget_url,
- jlong jadapter) {
+void AddSdchObserverOnNetworkThread(JNIEnv* env,
+ const JavaParamRef<jobject>& jsdch_observer,
+ const JavaParamRef<jstring>& jtarget_url,
+ jlong jcontext) {
base::android::ScopedJavaGlobalRef<jobject> jsdch_observer_ref;
// ScopedJavaGlobalRef do not hold onto the env reference, so it is safe to
// use it across threads. |AddSdchObserverHelper| will acquire a new
@@ -101,12 +92,9 @@ void AddSdchObserver(JNIEnv* env,
jsdch_observer_ref.Reset(env, jsdch_observer);
GURL target_url(base::android::ConvertJavaStringToUTF8(env, jtarget_url));
- CronetURLRequestContextAdapter* context_adapter =
- reinterpret_cast<CronetURLRequestContextAdapter*>(jadapter);
- context_adapter->PostTaskToNetworkThread(
- FROM_HERE,
- base::Bind(&AddSdchObserverOnNetworkThread, target_url,
- jsdch_observer_ref, base::Unretained(context_adapter)));
+ net::URLRequestContext* context =
+ reinterpret_cast<net::URLRequestContext*>(jcontext);
+ AddSdchObserverHelper(target_url, jsdch_observer_ref, context);
}
bool RegisterSdchTestUtil(JNIEnv* env) {

Powered by Google App Engine
This is Rietveld 408576698