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

Side by Side Diff: tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py

Issue 1044573002: Integration test for exp Chrome-Proxy request option (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import base64 5 import base64
6 import logging 6 import logging
7 import urlparse 7 import urlparse
8 8
9 from integration_tests import chrome_proxy_metrics as metrics 9 from integration_tests import chrome_proxy_metrics as metrics
10 from metrics import loading 10 from metrics import loading
(...skipping 292 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 def __init__(self): 303 def __init__(self):
304 super(ChromeProxyLoFi, self).__init__(restart_after_each_page=True) 304 super(ChromeProxyLoFi, self).__init__(restart_after_each_page=True)
305 305
306 def CustomizeBrowserOptions(self, options): 306 def CustomizeBrowserOptions(self, options):
307 super(ChromeProxyLoFi, self).CustomizeBrowserOptions(options) 307 super(ChromeProxyLoFi, self).CustomizeBrowserOptions(options)
308 options.AppendExtraBrowserArgs('--enable-data-reduction-proxy-lo-fi') 308 options.AppendExtraBrowserArgs('--enable-data-reduction-proxy-lo-fi')
309 309
310 def AddResults(self, tab, results): 310 def AddResults(self, tab, results):
311 self._metrics.AddResultsForLoFi(tab, results) 311 self._metrics.AddResultsForLoFi(tab, results)
312 312
313 class ChromeProxyExperiment(ChromeProxyValidation):
314 """Correctness measurement for experiment directives in Chrome-Proxy header.
315
316 This test verifies that experiment "test" in the Chrome-Proxy request header
sclittle 2015/04/03 01:34:48 Maybe reword 'experiment "test"' to '"exp=test"' h
megjablon 2015/04/07 21:18:40 Done.
317 causes a bypass on the experiment test page."""
318
319 def __init__(self):
320 super(ChromeProxyExperiment, self).__init__(restart_after_each_page=True)
321
322 def CustomizeBrowserOptions(self, options):
323 super(ChromeProxyExperiment, self).CustomizeBrowserOptions(options)
324 options.AppendExtraBrowserArgs('--data-reduction-proxy-experiment=test')
325
326 def AddResults(self, tab, results):
327 self._metrics.AddResultsForBypass(tab, results)
328
313 329
314 class ChromeProxyHTTPToDirectFallback(ChromeProxyValidation): 330 class ChromeProxyHTTPToDirectFallback(ChromeProxyValidation):
315 """Correctness measurement for HTTP proxy fallback to direct.""" 331 """Correctness measurement for HTTP proxy fallback to direct."""
316 332
317 def __init__(self): 333 def __init__(self):
318 super(ChromeProxyHTTPToDirectFallback, self).__init__( 334 super(ChromeProxyHTTPToDirectFallback, self).__init__(
319 restart_after_each_page=True) 335 restart_after_each_page=True)
320 336
321 def CustomizeBrowserOptions(self, options): 337 def CustomizeBrowserOptions(self, options):
322 super(ChromeProxyHTTPToDirectFallback, 338 super(ChromeProxyHTTPToDirectFallback,
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
382 self._metrics.AddResultsForDataSaving, 398 self._metrics.AddResultsForDataSaving,
383 ], 399 ],
384 'bypass': [self._metrics.AddResultsForBypass], 400 'bypass': [self._metrics.AddResultsForBypass],
385 } 401 }
386 if not self._page.name in page_to_metrics: 402 if not self._page.name in page_to_metrics:
387 raise page_test.MeasurementFailure( 403 raise page_test.MeasurementFailure(
388 'Invalid page name (%s) in smoke. Page name must be one of:\n%s' % ( 404 'Invalid page name (%s) in smoke. Page name must be one of:\n%s' % (
389 self._page.name, page_to_metrics.keys())) 405 self._page.name, page_to_metrics.keys()))
390 for add_result in page_to_metrics[self._page.name]: 406 for add_result in page_to_metrics[self._page.name]:
391 add_result(tab, results) 407 add_result(tab, results)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698