New issue
Advanced search Search tips

Issue 680576 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature

Blocking:
issue 679424



Sign in to add a comment

Implement the PassThrough Integration Test with ChromeDriver

Project Member Reported by robertogden@chromium.org, Jan 12 2017

Issue description

Correctness measurement for Chrome-Proxy-Accept-Transform identity directives. This test verifies that "identity" in the Chrome-Proxy-Accept-Transform request header causes a resource to be loaded without Data Reduction Proxy transformations

New File Name: smoke.py
New Method Name: testPassThrough

Telemetry PageSet: https://cs.chromium.org/chromium/src/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/pass_through.py
Telemetry Benchmark: https://cs.chromium.org/chromium/src/tools/chrome_proxy/integration_tests/chrome_proxy_benchmark.py?l=106
Telemetry Measurement: https://cs.chromium.org/chromium/src/tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py?q=file:tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py+"class+ChromeProxyPassThrough"
Telemetry Metric: https://cs.chromium.org/chromium/src/tools/chrome_proxy/integration_tests/chrome_proxy_metrics.py?q=file:tools/chrome_proxy/integration_tests/chrome_proxy_metrics.py+AddResultsForPassThrough

(This bug was automatically generated.)

 
Cc: robertogden@chromium.org
Labels: Integration-Test Pri-2 Type-Feature
Status: Available (was: Untriaged)

Comment 2 by bengr@chromium.org, Jan 23 2017

Owner: rajendrant@chromium.org
Status: Assigned (was: Available)

Comment 3 by bengr@chromium.org, Feb 23 2017

Labels: M-59
Owner: jiewu@chromium.org
Moving to jiewu@ who's interested in working on this.

Comment 5 by bengr@chromium.org, Apr 30 2017

Labels: -M-59 M-60

Comment 6 by jiewu@chromium.org, May 1 2017

Status: WontFix (was: Assigned)
We need a server side integration test instead of a client one for verifying "identity" in the Chrome-Proxy-Accept-Transform request header causes a resource to be loaded without Data Reduction Proxy transformations.

As a note though, we don't ever set Chrome-Proxy-Accept-Transform:identitiy in the client (instead CPAT is left off).

If it's not testing any client functionality, I think this would be better suited to be a server-side integration test.

Thus, I created the following bug: https://b.corp.google.com/issues/37858015

Sign in to add a comment