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

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

Issue 558333007: Setup initial mock url request job tests for Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: added jni binding for AddUrlInterceptors() Created 6 years, 3 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/cronet_tests_jni.cc
diff --git a/components/cronet/android/test/cronet_tests_jni.cc b/components/cronet/android/test/cronet_tests_jni.cc
index 88b2b8cf62041c896430ff1f6fec29f811d68fd1..b3610ea3da51ed72eb25a03558982737ff79f146 100644
--- a/components/cronet/android/test/cronet_tests_jni.cc
+++ b/components/cronet/android/test/cronet_tests_jni.cc
@@ -8,11 +8,13 @@
#include "base/android/jni_android.h"
#include "base/android/jni_registrar.h"
#include "components/cronet/android/cronet_loader.h"
+#include "mock_url_request_job_test.h"
namespace {
const base::android::RegistrationMethod kCronetTestsRegisteredMethods[] = {
- {"BaseAndroid", base::android::RegisterJni},
+ {"BaseAndroid", base::android::RegisterJni},
mef 2014/09/22 14:35:56 We actually don't need "baseandroid" here as it is
xunjieli 2014/09/22 15:56:23 Done. I see. Thanks! :)
+ {"MockURLRequestJobTest", cronet::RegisterMockUrlRequestJobTest},
};
} // namespace
@@ -35,6 +37,7 @@ extern "C" jint JNI_OnLoad(JavaVM* vm, void* reserved) {
arraysize(kCronetTestsRegisteredMethods))) {
return -1;
}
+
mef 2014/09/22 14:35:56 nit: spurious nl.
xunjieli 2014/09/22 15:56:23 Done.
return cronet_onload;
}

Powered by Google App Engine
This is Rietveld 408576698