New issue
Advanced search Search tips

Issue 663584 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Add integration test for data reduction proxy QUIC field trial with XT proxy

Project Member Reported by tbansal@chromium.org, Nov 9 2016

Issue description

There is already an integration test for data reduction proxy QUIC field trial where Chromium speaks to core data reduction proxy. Another integration tests should be added where Chromium is forced to speak QUIC to the XT proxy.
 
Components: Internals>Network>DataProxy
Labels: M-56
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/afa3ca6074fff2cc203eabf41fafc642cd9a8dc0

commit afa3ca6074fff2cc203eabf41fafc642cd9a8dc0
Author: tbansal <tbansal@chromium.org>
Date: Wed Nov 09 22:08:11 2016

Add a flag to disable the check if proxy is core data saver proxy

This CL adds a flag that disables the check if proxy is a core data
saver proxy. This flag would be later used in tests in conjunction
with setting the data saver proxy to an XT proxy to verify QUIC +
data saver communication with an XT proxy.

BUG= 663584 

Review-Url: https://codereview.chromium.org/2486093002
Cr-Commit-Position: refs/heads/master@{#431047}

[modify] https://crrev.com/afa3ca6074fff2cc203eabf41fafc642cd9a8dc0/components/data_reduction_proxy/core/browser/data_reduction_proxy_delegate.cc
[modify] https://crrev.com/afa3ca6074fff2cc203eabf41fafc642cd9a8dc0/components/data_reduction_proxy/core/common/data_reduction_proxy_switches.cc
[modify] https://crrev.com/afa3ca6074fff2cc203eabf41fafc642cd9a8dc0/components/data_reduction_proxy/core/common/data_reduction_proxy_switches.h

Blocking: -343579
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/02a0fe9ac80db197c640106ed6a36efbdec49e19

commit 02a0fe9ac80db197c640106ed6a36efbdec49e19
Author: tbansal <tbansal@chromium.org>
Date: Wed Mar 29 18:57:37 2017

Add integration test for QUIC with non core proxies

Other minor changes:
1. Separate out QUIC tests to quic.py.
2. Separate out QUIC test in two different tests.
3. Add chrome arg --data-reduction-proxy-enable-quic-on-non-core-proxies
to the tests so that QUIC usage would be enabled even when Chrome is
using a non-core proxy.

BUG= 663584 

Review-Url: https://codereview.chromium.org/2775173003
Cr-Commit-Position: refs/heads/master@{#460474}

[add] https://crrev.com/02a0fe9ac80db197c640106ed6a36efbdec49e19/tools/chrome_proxy/webdriver/quic.py
[modify] https://crrev.com/02a0fe9ac80db197c640106ed6a36efbdec49e19/tools/chrome_proxy/webdriver/smoke.py

Labels: -M-56 M-59
Status: Fixed (was: Started)

Sign in to add a comment