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

Unified Diff: tools/chrome_proxy/webdriver/lofi.py

Issue 2779973005: Fix errors when running against staging (Closed)
Patch Set: Add skip on lite page test Created 3 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
« no previous file with comments | « tools/chrome_proxy/webdriver/lite_page.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/chrome_proxy/webdriver/lofi.py
diff --git a/tools/chrome_proxy/webdriver/lofi.py b/tools/chrome_proxy/webdriver/lofi.py
index fca1f8566c3a7e8e586bce104cd64c7c6885f766..18935a068c2dff47ae1ebb8b436988340ca39042 100644
--- a/tools/chrome_proxy/webdriver/lofi.py
+++ b/tools/chrome_proxy/webdriver/lofi.py
@@ -45,6 +45,10 @@ class LoFi(IntegrationTest):
# load should not pick the Lo-Fi placeholder from cache and original image
# should be loaded.
def testLoFiCacheBypass(self):
+ # If it was attempted to run with another experiment, skip this test.
+ if common.ParseFlags().browser_args and ('--data-reduction-proxy-experiment'
+ in common.ParseFlags().browser_args):
+ self.skipTest('This test cannot be run with other experiments.')
with TestDriver() as test_driver:
# First page load, enable Lo-Fi and chrome proxy. Disable server
# experiments such as tamper detection. This test should be run with
« no previous file with comments | « tools/chrome_proxy/webdriver/lite_page.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698