Commit 794a66d4 authored by rajendrant's avatar rajendrant Committed by Commit Bot

Skip the quic test for <M76

For chrome which doesn't have the change to record the histograms
correctly, testCheckPageWithQuicProxyTransaction should be skipped.

Change-Id: I09e31acde881631859221a824fe858bb9c8b4d8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601342Reviewed-by: default avatarRobert Ogden <robertogden@chromium.org>
Commit-Queue: rajendrant <rajendrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657786}
parent f740c804
...@@ -7,6 +7,7 @@ import time ...@@ -7,6 +7,7 @@ import time
import common import common
from common import TestDriver from common import TestDriver
from common import IntegrationTest from common import IntegrationTest
from decorators import ChromeVersionEqualOrAfterM
class Quic(IntegrationTest): class Quic(IntegrationTest):
...@@ -35,6 +36,7 @@ class Quic(IntegrationTest): ...@@ -35,6 +36,7 @@ class Quic(IntegrationTest):
# Ensure Chrome uses QUIC DataSaver proxy when QUIC is enabled. This test # Ensure Chrome uses QUIC DataSaver proxy when QUIC is enabled. This test
# may fail if QUIC is disabled on the server side. # may fail if QUIC is disabled on the server side.
@ChromeVersionEqualOrAfterM(76)
def testCheckPageWithQuicProxyTransaction(self): def testCheckPageWithQuicProxyTransaction(self):
with TestDriver() as t: with TestDriver() as t:
t.AddChromeArg('--enable-spdy-proxy-auth') t.AddChromeArg('--enable-spdy-proxy-auth')
...@@ -53,6 +55,8 @@ class Quic(IntegrationTest): ...@@ -53,6 +55,8 @@ class Quic(IntegrationTest):
self.assertEqual(2, len(responses)) self.assertEqual(2, len(responses))
for response in responses: for response in responses:
self.assertHasProxyHeaders(response) self.assertHasProxyHeaders(response)
t.SleepUntilHistogramHasEntry('PageLoad.Clients.DataReductionProxy.'
'ParseTiming.NavigationToParseStart')
# Verify that histogram DataReductionProxy.Quic.ProxyStatus has at least 1 # Verify that histogram DataReductionProxy.Quic.ProxyStatus has at least 1
# sample. This sample must be in bucket 0 (QUIC_PROXY_STATUS_AVAILABLE). # sample. This sample must be in bucket 0 (QUIC_PROXY_STATUS_AVAILABLE).
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment