PlzNavigate: ensure POST navigations are treated as different document

PlzNavigate: ensure POST navigations are treated as different document

This CL fixes an issue where the FrameLoader in Blink would attempt to
commit a POST navigation as same-site even though a different document
navigation should (and has started to) be performed.

BUG=608372
Review-Url: https://codereview.chromium.org/2630413002
Cr-Commit-Position: refs/heads/master@{#446011}
Committed: https://chromium.googlesource.com/chromium/src/+/62c1bacb7df25e17142f3fd0a56c9a8965cde17d

[clamy, 2017-01-16 17:41:40]

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

[commit-bot: I haz the power, 2017-01-16 17:41:57]

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-01-16 19:02:36]

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

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

Dry run: This issue passed the CQ dry run.

[clamy, 2017-01-19 14:47:51]

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

[commit-bot: I haz the power, 2017-01-19 14:48:04]

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-01-19 16:24:34]

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

[commit-bot: I haz the power, 2017-01-19 16:24:35]

Dry run: This issue passed the CQ dry run.

[clamy, 2017-01-19 16:45:02]

Description was changed from

==========
PlzNavigate: ensure POST navigations are treated as different document

This CL fixes an issue where the FrameLoader in Blink would attempt to
commit a POST navigation as same-site even though a different document
navigation should (and has started to) be performed.

BUG=608372
==========

to

==========
PlzNavigate: ensure POST navigations are treated as different document

This CL fixes an issue where the FrameLoader in Blink would attempt to
commit a POST navigation as same-site even though a different document
navigation should (and has started to) be performed.

BUG=608372
==========

[clamy, 2017-01-19 16:45:02]

clamy@chromium.org changed reviewers:
+ japhet@chromium.org

[clamy, 2017-01-19 16:45:13]

@japhet: PTAL

[Nate Chapin, 2017-01-23 22:09:53]

Ok, this is specific to POST navigations that aren't labelled as form
submission? LGTM

[clamy, 2017-01-24 15:05:21]

Precisely. Thanks for the review!

[clamy, 2017-01-24 15:05:25]

The CQ bit was checked by clamy@chromium.org

[clamy, 2017-01-24 15:05:25]

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

[commit-bot: I haz the power, 2017-01-24 15:05:45]

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-01-24 15:09:06]

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

[commit-bot: I haz the power, 2017-01-24 15:09:07]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[clamy, 2017-01-25 12:56:36]

The CQ bit was checked by clamy@chromium.org

[commit-bot: I haz the power, 2017-01-25 12:56:51]

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-01-25 13:31:03]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485348996638950,
"parent_rev": "c782ea5b9c5d4e85c75ab8a2d601a088376cc1af", "commit_rev":
"62c1bacb7df25e17142f3fd0a56c9a8965cde17d"}

[commit-bot: I haz the power, 2017-01-25 13:31:37]

Description was changed from

==========
PlzNavigate: ensure POST navigations are treated as different document

This CL fixes an issue where the FrameLoader in Blink would attempt to
commit a POST navigation as same-site even though a different document
navigation should (and has started to) be performed.

BUG=608372
==========

to

==========
PlzNavigate: ensure POST navigations are treated as different document

This CL fixes an issue where the FrameLoader in Blink would attempt to
commit a POST navigation as same-site even though a different document
navigation should (and has started to) be performed.

BUG=608372

Review-Url: https://codereview.chromium.org/2630413002
Cr-Commit-Position: refs/heads/master@{#446011}
Committed:
https://chromium.googlesource.com/chromium/src/+/62c1bacb7df25e17142f3fd0a56c...
==========

[commit-bot: I haz the power, 2017-01-25 13:31:37]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/62c1bacb7df25e17142f3fd0a56c...