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

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

Issue 2775173003: Add integration test for QUIC with non core proxies (Closed)
Patch Set: ps 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/quic.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/smoke.py
diff --git a/tools/chrome_proxy/webdriver/smoke.py b/tools/chrome_proxy/webdriver/smoke.py
index b0a8991dd1e7adeb7b270c7fa0098cfa6b6f7220..f29dc235a850ca8dd162d0137cebaf392dd9f0bd 100644
--- a/tools/chrome_proxy/webdriver/smoke.py
+++ b/tools/chrome_proxy/webdriver/smoke.py
@@ -21,34 +21,6 @@ class Smoke(IntegrationTest):
t.LoadURL('http://check.googlezip.net/test.html')
for response in t.GetHTTPResponses():
self.assertNotHasChromeProxyViaHeader(response)
-
- # Ensure Chrome uses DataSaver with QUIC enabled.
- def testCheckPageWithQuicProxy(self):
- with TestDriver() as t:
- t.AddChromeArg('--enable-spdy-proxy-auth')
- t.AddChromeArg('--enable-quic')
- t.AddChromeArg('--data-reduction-proxy-http-proxies=https://proxy.googlezip.net:443')
- t.AddChromeArg('--force-fieldtrials=DataReductionProxyUseQuic/Enabled')
- t.LoadURL('http://check.googlezip.net/test.html')
- responses = t.GetHTTPResponses()
- self.assertEqual(2, len(responses))
- for response in responses:
- self.assertHasChromeProxyViaHeader(response)
-
- # Verify that histogram DataReductionProxy.Quic.ProxyStatus has at least 1
- # sample. This sample must be in bucket 0 (QUIC_PROXY_STATUS_AVAILABLE).
- proxy_status = t.GetHistogram('DataReductionProxy.Quic.ProxyStatus')
- self.assertLessEqual(1, proxy_status['count'])
- self.assertEqual(0, proxy_status['sum'])
-
- # Navigate to one more page to ensure that established QUIC connection
- # is used for the next request. Give 3 seconds extra headroom for the QUIC
- # connection to be established.
- time.sleep(3)
- t.LoadURL('http://check.googlezip.net/test.html')
- proxy_usage = t.GetHistogram('Net.QuicAlternativeProxy.Usage')
- # Bucket ALTERNATIVE_PROXY_USAGE_NO_RACE should have at least onesample.
- self.assertLessEqual(1, proxy_usage['buckets'][0]['count'])
# Ensure Chrome uses DataSaver in normal mode.
def testCheckPageWithNormalMode(self):
« no previous file with comments | « tools/chrome_proxy/webdriver/quic.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698