Add integration tests for QUIC

tools/chrome_proxy/integration_tests/chrome_proxy_metrics.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 json
 import logging
 import os
 import time
 
 from common import chrome_proxy_metrics
 from common import network_metrics
 from common.chrome_proxy_metrics import ChromeProxyMetricException
 from telemetry.page import page_test
 from telemetry.value import scalar
 from telemetry.value import histogram_util
@@ skipping 850 matching lines @@
           'Expected 1 pingback success and no failures, but '
           'there were %d succesful pingbacks and %d failed pingback attempts'
           % (succeeded, count - succeeded))
     results.AddValue(scalar.ScalarValue(
         results.current_page, 'attempted', 'count', attempted))
     results.AddValue(scalar.ScalarValue(
         results.current_page, 'succeeded_count', 'count', count))
     results.AddValue(scalar.ScalarValue(
         results.current_page, 'succeeded_sum', 'count', succeeded))
 
+  def AddResultsForQuicTransaction(self, tab, results):
+    histogram_type = histogram_util.BROWSER_HISTOGRAM
+    # This histogram should be synchronously created when the Navigate occurs.
+    # Verify that histogram DataReductionProxy.Quic.ProxyStatus has no samples
+    # in bucket >=1.
+    fail_counts_proxy_status = histogram_util.GetHistogramSum(
+        histogram_type,
+        'DataReductionProxy.Quic.ProxyStatus',
+        tab)
+    if fail_counts_proxy_status != 0:
+      raise ChromeProxyMetricException, (
+          'fail_counts_proxy_status is %d.' % fail_counts_proxy_status)
+
+    # Verify that histogram DataReductionProxy.Quic.ProxyStatus has at least 1
+    # sample. This sample must be in bucket 0 (QUIC_PROXY_STATUS_AVAILABLE).
+    success_counts_proxy_status = histogram_util.GetHistogramCount(
+        histogram_type,
+        'DataReductionProxy.Quic.ProxyStatus',
+        tab)
+    if success_counts_proxy_status <= 0:
+      raise ChromeProxyMetricException, (
+          'success_counts_proxy_status is %d.' % success_counts_proxy_status)
+
+    # Navigate to one more page to ensure that established QUIC connection
+    # is used for the next request.
+    time.sleep(3)

[RyanSturm, 2016/09/21 18:54:39]

Can you add a comment on why this sleep is 3 and why it is necessary? 3 seconds
is a large chunk of time for automated tests.

Pingback uses an incremental 10 second time that checks every 1 second. I'm not
sure if that approach is better for these tests or not though.

[tbansal1, 2016/09/22 17:25:42]

Done.

+    tab.Navigate('http://check.googlezip.net/test.html')
+
+    proxy_usage_histogram_json = histogram_util.GetHistogram(histogram_type,
+        'Net.QuicAlternativeProxy.Usage',
+        tab)
+    proxy_usage_histogram = json.loads(proxy_usage_histogram_json)
+
+    # Bucket ALTERNATIVE_PROXY_USAGE_NO_RACE should have at least one sample.
+    if proxy_usage_histogram['buckets'][0]['count'] <= 0:
+      raise ChromeProxyMetricException, (
+          'Number of samples in ALTERNATIVE_PROXY_USAGE_NO_RACE bucket is %d.'
+             % proxy_usage_histogram['buckets'][0]['count'])
+
+    results.AddValue(scalar.ScalarValue(
+        results.current_page, 'fail_counts_proxy_status', 'count',
+        fail_counts_proxy_status))
+    results.AddValue(scalar.ScalarValue(
+        results.current_page, 'success_counts_proxy_status', 'count',
+        success_counts_proxy_status))
+
   def AddResultsForBypassOnTimeout(self, tab, results):
     bypass_count = 0
     # Wait maximum of 120 seconds for test to complete. Should complete soon
     # after 90 second test server delay in case of failure, and much sooner in
     # case of success.
     tab.WaitForDocumentReadyStateToBeComplete(timeout=120)
     for resp in self.IterResponses(tab):
       if resp.HasChromeProxyViaHeader() and not resp.response.url.endswith(
           'favicon.ico'):
         r = resp.response
@@ skipping 152 matching lines @@
     except KeyError:
       raise ChromeProxyMetricException, 'No metrics found'
 
 # Returns whether |url| is a block-once test URL. Data Reduction Proxy has been
 # configured to always return block-once for these URLs.
 def IsTestUrlForBlockOnce(url):
   return (url == 'http://check.googlezip.net/blocksingle/' or
       url == ('http://chromeproxy-test.appspot.com/default?respBody=T0s='
               '&respHeader=eyJBY2Nlc3MtQ29udHJvbC1BbGxvdy1PcmlnaW4iOlsiKiJ'
               'dfQ==&respStatus=200&flywheelAction=block-once'))