Closed
Bug 1307779
Opened 8 years ago
Closed 8 years ago
Pressing volume down shows tab history panel
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(firefox49 unaffected, fennec50+, firefox50+ verified, firefox51+ verified, firefox52+ verified)
VERIFIED
FIXED
Firefox 52
People
(Reporter: sebastian, Assigned: ahunt)
References
Details
(Keywords: regression, Whiteboard: [MobileAS])
Attachments
(1 file)
58 bytes,
text/x-review-board-request
|
sebastian
:
review+
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details |
I assume this is a regression from bug 1304688: When pressing the hardware volume down button the tab history panel shows up (Nexus 6P, Android 7.0).
Assignee | ||
Comment 2•8 years ago
|
||
(In reply to Sebastian Kaspari (:sebastian) from comment #1) > @ahunt: Are you seeing this too? Yes. And I realise now that I forgot to test that it's the back key that's being pressed in onKeyDown() : /.
Assignee: nobody → ahunt
Flags: needinfo?(ahunt)
Priority: -- → P1
Whiteboard: [MobileAS]
Assignee | ||
Comment 3•8 years ago
|
||
I filed Bug 1307836 to add tests for this, but I'm not sure how feasible adding tests actually is (we'd need to setup emulators for multiple android versions...). With my patch (see next comment), I've confirmed that both on an Android 7 (N) and an Android 4.4 device: - Single press back button works as expected - Long press shows history - Pressing volume up / down dowsn't show history
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Assignee | ||
Comment 5•8 years ago
|
||
[Tracking Requested - why for this release]: Regression introduced in Bug 1304688, affects all Android 7 devices: we'll show the history panel for all button presses (primarily volume down/up).
tracking-firefox50:
--- → ?
tracking-firefox51:
--- → ?
Reporter | ||
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8798130 [details] Bug 1307779 - Only start long-press timer for back button https://reviewboard.mozilla.org/r/83676/#review82242
Attachment #8798130 -
Flags: review?(s.kaspari) → review+
Updated•8 years ago
|
tracking-fennec: --- → ?
status-firefox49:
--- → unaffected
status-firefox50:
--- → affected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
tracking-firefox52:
--- → ?
Updated•8 years ago
|
tracking-fennec: ? → 50+
Pushed by ahunt@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/23b4707ed4f9 Only start long-press timer for back button r=sebastian
Comment 8•8 years ago
|
||
Tracking because it is a new regression which never shipped to our users. Andrzej, Could you fill the uplift request? Thanks
Flags: needinfo?(ahunt)
Keywords: regression
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/23b4707ed4f9
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Reporter | ||
Comment 10•8 years ago
|
||
Can you request uplift for the release channels we uplifted the patch in bug 1304688 to?
Assignee | ||
Comment 11•8 years ago
|
||
Comment on attachment 8798130 [details] Bug 1307779 - Only start long-press timer for back button Approval Request Comment [Feature/regressing bug #]: Bug 1304688 introduced this regression, while trying to work around an Android 7 system bug. [User impact if declined]: Only on Android 7: pressing volume up/down (or any other hardware button other than the back button) results in the tab-history panel being shown after a ~3s delay. [Describe test coverage new/current, TreeHerder]: manual testing on nightly, on both older (Android <=6) and newer (Android 7) devices. [Risks and why]: Low risk: an additional condition has been added to the test for starting the backpress timer, this ensures that we only start the timer for the back button (but not other buttons). [String/UUID change made/needed]: none.
Flags: needinfo?(ahunt)
Attachment #8798130 -
Flags: approval-mozilla-beta?
Attachment #8798130 -
Flags: approval-mozilla-aurora?
Comment on attachment 8798130 [details] Bug 1307779 - Only start long-press timer for back button Fixes a new regression in 50, Aurora51+, Beta50+
Attachment #8798130 -
Flags: approval-mozilla-beta?
Attachment #8798130 -
Flags: approval-mozilla-beta+
Attachment #8798130 -
Flags: approval-mozilla-aurora?
Attachment #8798130 -
Flags: approval-mozilla-aurora+
Comment 13•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/637b73dad0ae
Comment 14•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/c8c31aad61a1
Comment 15•8 years ago
|
||
Verified this on Nexus 6P(Android 7.0) on Firefox 50 Beta, latest Aurora and latest Nightly
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Iteration: --- → 1.6
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•