Clear stale NavigationParams from HistoryController.

Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/4d78d33cb1852e7d487f1d327517c4c4044a3ca3
Cr-Commit-Position: refs/heads/master@{#404870}

[Charlie Reis, 2016-07-12 20:55:10]

Description was changed from

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
==========

to

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Charlie Reis, 2016-07-12 21:06:16]

creis@chromium.org changed reviewers:
+ avi@chromium.org

[Charlie Reis, 2016-07-12 21:06:16]

Avi, can you review?  This is a followup to
https://codereview.chromium.org/2134493002, based on comments 18-22 on the bug.

Hoping this gets us by until we remove HistoryController in the new navigation
path, since it doesn't really track enough state to get things 100% right.

[Charlie Reis, 2016-07-12 21:12:05]

Description was changed from

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Charlie Reis, 2016-07-12 21:12:06]

creis@chromium.org changed reviewers:
+ nick@chromium.org

[ncarter (slow), 2016-07-12 21:16:26]

lgtm

[Charlie Reis, 2016-07-12 21:28:34]

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

[commit-bot: I haz the power, 2016-07-12 21:35:44]

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

[Charlie Reis, 2016-07-12 21:43:43]

The CQ bit was unchecked by creis@chromium.org

[Charlie Reis, 2016-07-12 21:43:50]

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

[commit-bot: I haz the power, 2016-07-12 21:45:10]

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

[Avi (use Gerrit), 2016-07-12 23:17:13]

lgtm

[Charlie Reis, 2016-07-12 23:17:54]

The CQ bit was unchecked by creis@chromium.org

[Charlie Reis, 2016-07-12 23:20:28]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2016-07-12 23:22:54]

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, 2016-07-13 01:13:19]

Description was changed from

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-07-13 01:13:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 01:15:24]

Description was changed from

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Clear stale NavigationParams from HistoryController.

This prevents newly created iframes during a back/forward from
targeting the wrong NavigationEntry.

BUG=623319
TEST=See bug comment 20 for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/4d78d33cb1852e7d487f1d327517c4c4044a3ca3
Cr-Commit-Position: refs/heads/master@{#404870}
==========

[commit-bot: I haz the power, 2016-07-13 01:15:25]

Patchset 1 (id:??) landed as
https://crrev.com/4d78d33cb1852e7d487f1d327517c4c4044a3ca3
Cr-Commit-Position: refs/heads/master@{#404870}