Closed
Bug 1698565
Opened 4 years ago
Closed 4 years ago
Actually do a target switch in devtools/client/framework/test/browser_toolbox_backward_forward_navigation.js
Categories
(DevTools :: Framework, defect)
DevTools
Framework
Tracking
(firefox88 fixed)
RESOLVED
FIXED
88 Branch
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
The test was navigating between the same origin
Assignee | ||
Comment 1•4 years ago
|
||
The test wasn't doing a target switch, which it was supposed to do, so this patch
fixes that.
The other issue was that in the case we're navigating back and forth immediately,
we might get into cases where the goForward
command was ignored. The fix here
is to use waitForLocationChange
which does wait just enough so the navigation
was taken into account.
Depends on D107980
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/363675068fc2 [devtools] Fix browser_toolbox_backward_forward_navigation.js. r=ochameau.
Comment 3•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox88:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•