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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java

Issue 1751183002: Precache uses GcmNetworkManager for task scheduling (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments and fixed tests Created 4 years, 9 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: chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java
index 9a753c3a6ee1e8787011bb283a803874256e85fe..84c8b5e1faa1c54fe221553dbea6478fc013638e 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.java
@@ -66,7 +66,6 @@ public class PrecacheLauncherTest extends NativeLibraryTestBase {
@Override
protected void setUp() throws Exception {
super.setUp();
-
// This is a PrecacheLauncher with a stubbed out nativeShouldRun so we can change that on
// the fly without needing to set up a sync backend.
mLauncher = new PrecacheLauncherUnderTest();
@@ -74,7 +73,7 @@ public class PrecacheLauncherTest extends NativeLibraryTestBase {
// The target context persists throughout the entire test run, and so leaks state between
// tests. We reset the is_precaching_enabled pref to false to make the test run consistent,
// in case another test class has modified this pref.
- PrecacheServiceLauncher.setIsPrecachingEnabled(getTargetContext(), false);
+ PrecacheController.setIsPrecachingEnabled(getTargetContext(), false);
// ProfileSyncService needs the browser process to be running.
loadNativeLibraryAndInitBrowserProcess();
@@ -95,7 +94,7 @@ public class PrecacheLauncherTest extends NativeLibraryTestBase {
@Override
protected void tearDown() throws Exception {
ProfileSyncService.overrideForTests(null);
- PrecacheServiceLauncher.setIsPrecachingEnabled(getTargetContext(), false);
+ PrecacheController.setIsPrecachingEnabled(getTargetContext(), false);
super.tearDown();
}
@@ -192,7 +191,7 @@ public class PrecacheLauncherTest extends NativeLibraryTestBase {
/** Return the value of the is_precaching_enabled pref, as set by updateEnabledSync. */
private boolean isPrecachingEnabled() {
- return PrecacheServiceLauncher.isPrecachingEnabled(getTargetContext());
+ return PrecacheController.get(getTargetContext()).isPrecachingEnabled();
}
/** Return the set of failure reasons for mLauncher. */

Powered by Google App Engine
This is Rietveld 408576698