Integration test for data reduction proxy version header

tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import base64
 import logging
 import urlparse
 
 from integration_tests import chrome_proxy_metrics as metrics
 from metrics import loading
@@ skipping 196 matching lines @@
 
   def AddResults(self, tab, results):
     proxies = [
         _TEST_SERVER + ":80",
         self._metrics.effective_proxies['fallback'],
         self._metrics.effective_proxies['direct']]
     bad_proxies = [_TEST_SERVER + ":80"]
     self._metrics.AddResultsForHTTPFallback(tab, results, proxies, bad_proxies)
 
 
+    # Depends on the fix of http://crbug.com/330342.

[bolian, 2014/09/04 02:09:02]

wrong comment?

[bengr, 2014/09/04 18:42:23]

Done.

+class ChromeProxyVersionTest(ChromeProxyValidation):
+  """Correctness measurement for version directives in Chrome-Proxy header.
+
+  In this test, the configured proxy is the chromeproxy-test server. The test

[bolian, 2014/09/04 02:09:02]

rm the first sentence.

[bengr, 2014/09/04 18:42:23]

Done.

+  verifies that the version information provided in the Chrome-Proxy request
+  header overrides any version, if specified, that is provided in the user
+  agent string.
+  """
+
+  def __init__(self):
+    super(ChromeProxyVersionTest, self).__init__()
+
+  def CustomizeBrowserOptions(self, options):
+    super(ChromeProxyVersionTest,
+          self).CustomizeBrowserOptions(options)
+    options.AppendExtraBrowserArgs('--user-agent="No Version"')

[bolian, 2014/09/04 02:09:02]

This is not the final UA string, right?  You want to set an older version when
the server change rolled out?

[bengr, 2014/09/04 18:42:23]

Done.

+
+  def AddResults(self, tab, results):
+    self._metrics.AddResultsForVersionTest(tab, results)
+
+
 class ChromeProxySmoke(ChromeProxyValidation):
   """Smoke measurement for basic chrome proxy correctness."""
 
   def __init__(self):
     super(ChromeProxySmoke, self).__init__()
 
   def WillNavigateToPage(self, page, tab):
     super(ChromeProxySmoke, self).WillNavigateToPage(page, tab)
     if page.name == 'safebrowsing':
       self._expect_timeout = True
@@ skipping 16 matching lines @@
             ],
         'bypass': [self._metrics.AddResultsForBypass],
         'safebrowsing': [self._metrics.AddResultsForSafebrowsing],
         }
     if not self._page.name in page_to_metrics:
       raise page_test.MeasurementFailure(
           'Invalid page name (%s) in smoke. Page name must be one of:\n%s' % (
           self._page.name, page_to_metrics.keys()))
     for add_result in page_to_metrics[self._page.name]:
       add_result(tab, results)