Mouse Back/Forward break in Responsive Design Mode
Categories
(DevTools :: Responsive Design Mode, defect, P2)
Tracking
(firefox-esr68 unaffected, firefox-esr78 disabled, firefox77 wontfix, firefox78 disabled, firefox79 verified)
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox-esr78 | --- | disabled |
firefox77 | --- | wontfix |
firefox78 | --- | disabled |
firefox79 | --- | verified |
People
(Reporter: lancehmd, Assigned: mtigley)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-release-
|
Details |
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-release-
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:77.0) Gecko/20100101 Firefox/77.0
Steps to reproduce:
Open Responsive Design Mode (Ctrl+Shift+M), try to use Back or Forward buttons on a mouse.
Actual results:
Nothing.
Expected results:
Go one page forward or one page back.
Comment 1•4 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Assignee | ||
Comment 2•4 years ago
|
||
Thank you for reporting this issue! I can confirm this is also reproducible on Firefox Nightly 79 and Firefox Developer Edition 78 when the pref: devtools.responsive.browserUI.enabled = false
. I'll investigate.
Assignee | ||
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D78835
Comment 6•4 years ago
|
||
Backed out 2 changesets (Bug 1644248) for causing dt failures in browser_old-rdm_navigation.js
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=305724192&repo=autoland&lineNumber=1099
Backout: https://hg.mozilla.org/integration/autoland/rev/247bc42758fad4a6f190dc9eafcedbc8e1382847
Assignee | ||
Updated•4 years ago
|
Comment 9•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/573a96c80046
https://hg.mozilla.org/mozilla-central/rev/093ce2a6ea7b
Assignee | ||
Comment 10•4 years ago
|
||
Comment on attachment 9155156 [details]
Bug 1644248 - Fallback on RemoteWebNavigation's method for navigating backward/forward.
Beta/Release Uplift Approval Request
- User impact if declined: Navigating forward/back in RDM will not work for users and will make the tool appear broken.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: comment 0
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Patch contains small and relatively trivial changes.
- String changes made/needed: no
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 11•4 years ago
|
||
Reproduced the initial issue in Beta 78.0b5 (build id: 20200609215727).
Verified - Fixed in latest Nightly build 79.0a1 (build id: 20200611093454) using Windows 10 and Ubuntu 18.04.
Comment 12•4 years ago
|
||
I guess you meant to request uplift to beta (78)? Is this a regression? What caused it?
Assignee | ||
Comment 13•4 years ago
|
||
(In reply to Julien Cristau [:jcristau] from comment #12)
I guess you meant to request uplift to beta (78)? Is this a regression? What caused it?
Would it be possible to uplift the fix to release (77)? This issue is only reproducible when the pref devtools.responsive.browserUI.enabled
is false and Firefox Release 77 is the only build where this is the case.
I was also able to narrow the regression to Bug 1614462. So this issue has been around for a few months already and our tests were not sufficient enough to catch this. The tests added here will fix this.
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 14•4 years ago
|
||
Comment on attachment 9155156 [details]
Bug 1644248 - Fallback on RemoteWebNavigation's method for navigating backward/forward.
No, we just merged 78 to mozilla-release ahead of next week's release, so 77 won't see an update anymore.
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Description
•