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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java

Issue 1772233003: Flag for Background Loading of Offline Pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Put @VisibleForTesting in the right place. 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/offlinepages/OfflinePageBridgeTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java
index 78f819e1da22a2880a8a8e7de9a43086bc7c1e1f..dcdb1dda1535a07cab1b007b46577a94597c57b7 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridgeTest.java
@@ -258,6 +258,14 @@ public class OfflinePageBridgeTest extends ChromeActivityTestCaseBase<ChromeActi
mOfflinePageBridge.getOfflineUrlForOnlineUrl(offlinePage.getUrl()));
}
+ @CommandLineFlags.Add("disable-features=offline-pages-background-loading")
+ @SmallTest
+ public void testBackgroundLoadSwitch() throws Exception {
+ // We should be able to call the C++ is enabled function from the Java side.
+ assertFalse("If background loading is off, we should see the feature disabled",
+ OfflinePageBridge.isBackgroundLoadingEnabled());
+ }
+
private void savePage(final int expectedResult, final String expectedUrl)
throws InterruptedException {
final Semaphore semaphore = new Semaphore(0);

Powered by Google App Engine
This is Rietveld 408576698