PlzNavigate: Fix "Only one CSP is reported instead of several".

PlzNavigate: Fix "Only one CSP is reported instead of several".

This CL makes that every infringed policy are reported instead of the
infringed first one. One unit test is added as well.

TEST=external/wpt/content-security-policy/frame-src/frame-src-redirect.html

BUG=723602
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel;master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2893613002
Cr-Commit-Position: refs/heads/master@{#473580}
Committed: https://chromium.googlesource.com/chromium/src/+/2c1e5408c1f8e69ae1891edb22a8502b13880846

[arthursonzogni, 2017-05-17 14:43:01]

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

[commit-bot: I haz the power, 2017-05-17 14:43:36]

Dry run: CQ is trying da patch.

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

[arthursonzogni, 2017-05-17 14:49:28]

Description was changed from

==========
PlzNavigate: Fix "Only one CSP is reported instead of several".

This CL makes that every infringed policy are reported instead of the
infringed first one. One unit test is added as well.

TEST=external/wpt/content-security-policy/frame-src/frame-src-redirect.html

BUG=723602
==========

to

==========
PlzNavigate: Fix "Only one CSP is reported instead of several".

This CL makes that every infringed policy are reported instead of the
infringed first one. One unit test is added as well.

TEST=external/wpt/content-security-policy/frame-src/frame-src-redirect.html

BUG=723602
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel;master.tryserver.chromium.linux:linux_site_isolation
==========

[arthursonzogni, 2017-05-17 14:49:40]

The CQ bit was unchecked by arthursonzogni@chromium.org

[arthursonzogni, 2017-05-17 14:50:18]

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

[commit-bot: I haz the power, 2017-05-17 14:50:48]

Dry run: CQ is trying da patch.

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

[arthursonzogni, 2017-05-17 15:12:36]

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

[commit-bot: I haz the power, 2017-05-17 15:13:08]

Dry run: CQ is trying da patch.

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

[arthursonzogni, 2017-05-17 16:31:51]

arthursonzogni@chromium.org changed reviewers:
+ mkwst@chromium.org

[arthursonzogni, 2017-05-17 16:31:52]

Hi Mike,

Please take a look.

[commit-bot: I haz the power, 2017-05-17 17:05:08]

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

[commit-bot: I haz the power, 2017-05-17 17:05:09]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Mike West, 2017-05-22 08:15:14]

LGTM!

[arthursonzogni, 2017-05-22 10:03:37]

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

[commit-bot: I haz the power, 2017-05-22 10:03:53]

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-05-22 12:44:35]

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

[commit-bot: I haz the power, 2017-05-22 12:44:36]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[arthursonzogni, 2017-05-22 14:22:23]

Thanks!

[arthursonzogni, 2017-05-22 14:22:38]

The CQ bit was checked by arthursonzogni@chromium.org

[arthursonzogni, 2017-05-22 14:22:38]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/2893613002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-05-22 14:22:55]

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-05-22 15:33:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495462958149660,
"parent_rev": "863dc7c7934fb53d57d29173a5c99a47f3a7bac6", "commit_rev":
"2c1e5408c1f8e69ae1891edb22a8502b13880846"}

[commit-bot: I haz the power, 2017-05-22 15:34:10]

Description was changed from

==========
PlzNavigate: Fix "Only one CSP is reported instead of several".

This CL makes that every infringed policy are reported instead of the
infringed first one. One unit test is added as well.

TEST=external/wpt/content-security-policy/frame-src/frame-src-redirect.html

BUG=723602
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel;master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
PlzNavigate: Fix "Only one CSP is reported instead of several".

This CL makes that every infringed policy are reported instead of the
infringed first one. One unit test is added as well.

TEST=external/wpt/content-security-policy/frame-src/frame-src-redirect.html

BUG=723602
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_browser_side_navigation_rel;master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2893613002
Cr-Commit-Position: refs/heads/master@{#473580}
Committed:
https://chromium.googlesource.com/chromium/src/+/2c1e5408c1f8e69ae1891edb22a8...
==========

[commit-bot: I haz the power, 2017-05-22 15:34:19]

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