New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 724271 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Add test for is_history_navigation_in_new_child logic in OnBeginNavigation

Project Member Reported by creis@chromium.org, May 18 2017

Issue description

jam@ pointed out that no tests are hitting the OnBeginNavigation logic ananta@ added in https://codereview.chromium.org/2701093002/.  However, as I determined in https://codereview.chromium.org/2890613002/#msg15, that code is critical to preventing a regression of  issue 657896  in some race outcomes.

We should try to add a content_browsertest that covers this branch of the race, where the new JS navigation arrives in OnBeginNavigation before the history navigation gets to PlzNavigate's CommitNavigation stage.
 

Sign in to add a comment