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

Issue 691583 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 791981
Owner:
Closed: Dec 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Double check we have tests for BrowseAccessibilityManager::UserIsReloading

Project Member Reported by jam@chromium.org, Feb 13 2017

Issue description

From the comment in https://codereview.chromium.org/2684143002/diff/80001/content/browser/web_contents/web_contents_impl.cc#newcode3312

Dominic mentioned that this area doesn't have test coverage. I've kept the old behavior of calling the BAM method on the old RFHI, but it would be good to have tests for this to show that the call has to be done on it and not the new one (and does it actually matter in practice?)
 
Blockedon: 791981
Mergedinto: 791981
Status: Duplicate (was: Assigned)
Covered by NavigationAccessibilityTest.TestNavigateToNewUrl, but that test is disabled

Blockedon: -791981

Sign in to add a comment