DescriptionOOPIF: Fix subframe back/forward after recreating FTNs (try #2).
In some cases, we would navigate the main frame when going back/forward
in a subframe. This was because the FTN ID had changed from what was
stored in the FrameNavigationEntry. We now use the frame's unique name
for subframes and the position in the tree for main frames, as is done
in default Chrome (in HistoryController).
This also relaxes the restriction that a FrameNavigationEntry's item
and document sequence numbers don't change, at least until
https://crbug.com/596707 is fixed.
This is a second attempt, after r386756 caused issue 603245.
BUG=586324, 568768
TEST=See bug for repro steps.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
Committed: https://crrev.com/22a7b4c06a8968ac455072e002af7abdc911c2ad
Cr-Commit-Position: refs/heads/master@{#390317}
Patch Set 1 : Try #1, rebased #Patch Set 2 : Remove FTN ID from FNE #Patch Set 3 : Fix tests #Patch Set 4 : Clean up and add tests #
Total comments: 16
Patch Set 5 : Rebase #Patch Set 6 : Fix nits #Messages
Total messages: 13 (6 generated)
|