Adding a CORS bypass integration test

Adding a CORS bypass integration test

This test verifies that CORS requests are treated with block-once by the
data reduction proxy server.

BUG=680556
Review-Url: https://codereview.chromium.org/2661363002
Cr-Commit-Position: refs/heads/master@{#447422}
Committed: https://chromium.googlesource.com/chromium/src/+/7d56868afb2fd253cfe318d2e39fe591a458f0db

[RyanSturm, 2017-01-31 21:32:46]

ryansturm@chromium.org changed reviewers:
+ megjablon@chromium.org, robertogden@chromium.org

[RyanSturm, 2017-01-31 21:32:46]

megjablon, robertogden: PTAL, thanks

[Robert Ogden, 2017-01-31 22:10:26]

lgtm

[megjablon, 2017-02-01 00:14:38]

https://codereview.chromium.org/2661363002/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/cors_bypass.py (right):

https://codereview.chromium.org/2661363002/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/cors_bypass.py:1: # Copyright 2017 The Chromium
Authors. All rights reserved.
The bug and spreadsheet said this should go in bypass.py. Robert, do we want
separate files for these or aggregated as you organized?

[RyanSturm, 2017-02-01 01:44:55]

The CQ bit was checked by ryansturm@chromium.org to run a CQ dry run

[RyanSturm, 2017-02-01 01:45:42]

megjablon: PTAL

https://codereview.chromium.org/2661363002/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/cors_bypass.py (right):

https://codereview.chromium.org/2661363002/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/cors_bypass.py:1: # Copyright 2017 The Chromium
Authors. All rights reserved.
On 2017/02/01 00:14:38, megjablon wrote:
> The bug and spreadsheet said this should go in bypass.py. Robert, do we want
> separate files for these or aggregated as you organized?

Done.

[commit-bot: I haz the power, 2017-02-01 01:45:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-01 01:56:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-01 01:56:48]

Dry run: This issue passed the CQ dry run.

[megjablon, 2017-02-01 01:57:52]

lgtm

[RyanSturm, 2017-02-01 02:01:12]

The CQ bit was checked by ryansturm@chromium.org

[RyanSturm, 2017-02-01 02:01:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertogden@chromium.org
Link to the patchset: https://codereview.chromium.org/2661363002/#ps20001
(title: "moving to bypass.py")

[commit-bot: I haz the power, 2017-02-01 02:01:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-01 02:35:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485914472672590,
"parent_rev": "2c1516d6d152e750ee6b0cabe3e1e032e20b3c1b", "commit_rev":
"7d56868afb2fd253cfe318d2e39fe591a458f0db"}

[commit-bot: I haz the power, 2017-02-01 02:36:32]

Description was changed from

==========
Adding a CORS bypass integration test

This test verifies that CORS requests are treated with block-once by the
data reduction proxy server.

BUG=680556
==========

to

==========
Adding a CORS bypass integration test

This test verifies that CORS requests are treated with block-once by the
data reduction proxy server.

BUG=680556

Review-Url: https://codereview.chromium.org/2661363002
Cr-Commit-Position: refs/heads/master@{#447422}
Committed:
https://chromium.googlesource.com/chromium/src/+/7d56868afb2fd253cfe318d2e39f...
==========

[commit-bot: I haz the power, 2017-02-01 02:36:33]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7d56868afb2fd253cfe318d2e39f...