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

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java

Issue 2686203002: [Offline pages] Creating BackgroundJobScheduler, which uses JobScheduler (Closed)
Patch Set: Adding null checks Created 3 years, 10 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/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java
index 0e22addef265b002d64146cbe5a5164554d44703..dd11396376ebe6f02835383e4a3ff182bf230df3 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundSchedulerTest.java
@@ -39,7 +39,7 @@ public class BackgroundSchedulerTest {
@Before
public void setUp() throws Exception {
- mContext = RuntimeEnvironment.application;
+ mContext = RuntimeEnvironment.application;
mGcmNetworkManager = (ShadowGcmNetworkManager) ShadowExtractor.extract(
GcmNetworkManager.getInstance(mContext));
mGcmNetworkManager.clear();
@@ -49,7 +49,7 @@ public class BackgroundSchedulerTest {
@Feature({"OfflinePages"})
public void testSchedule() {
assertNull(mGcmNetworkManager.getScheduledTask());
- BackgroundScheduler.schedule(mContext, mConditions1);
+ BackgroundScheduler.getInstance(mContext).schedule(mConditions1);
// Check with gcmNetworkManagerShadow that schedule got called.
assertNotNull(mGcmNetworkManager.getScheduledTask());
@@ -64,13 +64,13 @@ public class BackgroundSchedulerTest {
@Test
@Feature({"OfflinePages"})
- public void testUnschedule() {
+ public void testCancel() {
assertNull(mGcmNetworkManager.getScheduledTask());
- BackgroundScheduler.schedule(mContext, mConditions1);
+ BackgroundScheduler.getInstance(mContext).schedule(mConditions1);
assertNotNull(mGcmNetworkManager.getScheduledTask());
assertNull(mGcmNetworkManager.getCanceledTask());
- BackgroundScheduler.unschedule(mContext);
+ BackgroundScheduler.getInstance(mContext).cancel();
assertNotNull(mGcmNetworkManager.getCanceledTask());
}
}

Powered by Google App Engine
This is Rietveld 408576698