Remove "client_version" test from chromium

I cleaned up all the references in our various waterfalls, and this will
remove the test completely.

BUG=578272

Review URL: https://codereview.chromium.org/1640483002

Cr-Commit-Position: refs/heads/master@{#371678}

tools/chrome_proxy/integration_tests/chrome_proxy_benchmark.py

@@ -10,16 +10,6 @@ from telemetry import benchmark
 DESKTOP_PLATFORMS = ['mac', 'linux', 'win', 'chromeos']
 WEBVIEW_PLATFORMS = ['android-webview', 'android-webview-shell']
 
-class ChromeProxyClientVersion(ChromeProxyBenchmark):
-  tag = 'client_version'
-  test = measurements.ChromeProxyClientVersion
-  page_set = pagesets.SyntheticStorySet
-
-  @classmethod
-  def Name(cls):
-    return 'chrome_proxy_benchmark.client_version.synthetic'
-
-
 class ChromeProxyClientType(ChromeProxyBenchmark):
   tag = 'client_type'
   test = measurements.ChromeProxyClientType

tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py

@@ -241,27 +241,6 @@ class ChromeProxyHTTPFallbackViaHeader(ChromeProxyValidation):
     self._metrics.AddResultsForHTTPFallback(tab, results)
 
 
-class ChromeProxyClientVersion(ChromeProxyValidation):
-  """Correctness measurement for version directives in Chrome-Proxy header.
-
-  The test 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(ChromeProxyClientVersion, self).__init__(
-        metrics=metrics.ChromeProxyMetric())
-
-  def CustomizeBrowserOptions(self, options):
-    super(ChromeProxyClientVersion,
-          self).CustomizeBrowserOptions(options)
-    options.AppendExtraBrowserArgs('--user-agent="Chrome/32.0.1700.99"')
-
-  def AddResults(self, tab, results):
-    self._metrics.AddResultsForClientVersion(tab, results)
-
-
 class ChromeProxyClientType(ChromeProxyValidation):
   """Correctness measurement for Chrome-Proxy header client type directives."""
 

tools/chrome_proxy/integration_tests/chrome_proxy_metrics.py

@@ -170,25 +170,6 @@ class ChromeProxyMetric(network_metrics.NetworkMetric):
     results.AddValue(scalar.ScalarValue(
         results.current_page, 'extra_via_header', 'count', extra_via_count))
 
-  def AddResultsForClientVersion(self, tab, results):
-    via_count = 0
-    for resp in self.IterResponses(tab):
-      r = resp.response
-      if resp.response.status != 200:
-        raise ChromeProxyMetricException, ('%s: Response is not 200: %d' %
-                                           (r.url, r.status))
-      if not resp.IsValidByViaHeader():
-        raise ChromeProxyMetricException, ('%s: Response missing via header' %
-                                           (r.url))
-      via_count += 1
-
-    if via_count == 0:
-      raise ChromeProxyMetricException, (
-          'Expected at least one response through the proxy, but zero such '
-          'responses were received.')
-    results.AddValue(scalar.ScalarValue(
-        results.current_page, 'responses_via_proxy', 'count', via_count))
-
   def GetClientTypeFromRequests(self, tab):
     """Get the Chrome-Proxy client type value from requests made in this tab.