Closed
Bug 659402
Opened 13 years ago
Closed 13 years ago
Fullscreen video controls are broken
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox6 fixed, fennec6+)
VERIFIED
FIXED
Firefox 6
People
(Reporter: wesj, Assigned: wesj)
References
Details
(Keywords: regression)
Attachments
(1 file)
594 bytes,
text/plain
|
mbrubeck
:
review+
mfinkle
:
approval-mozilla-aurora+
|
Details |
Touch events shifted us from using Browser:MouseUp messages for clicks to Browser:MouseClick. Unfortunately, video controls redispatch their own events, and forgot to come to the party. Patch updates them.
Assignee | ||
Updated•13 years ago
|
Attachment #534854 -
Flags: review?(mbrubeck)
Updated•13 years ago
|
tracking-fennec: --- → 6+
Comment 1•13 years ago
|
||
we'll need this on aurora, so ask for approval after you get an r+
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #534854 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 534854 [details]
Patch
Asking for aurora approval
Attachment #534854 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 3•13 years ago
|
||
Just to note for the aurora approval, this patch is very low risk (code only runs on Fennec, Fullscreen video mode) on and fixes a regression we introduced when we checked in touchevents. Without it, it is impossible to control of exit fullscreen video requiring the user to quit fennec (hard to do) to get back to normal use.
Comment 4•13 years ago
|
||
Comment on attachment 534854 [details]
Patch
let this bake on trunk for a day or two before landing on aurora
Attachment #534854 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 5•13 years ago
|
||
Hmm... did I forget to put the first push in here? mc: http://hg.mozilla.org/mozilla-central/rev/24f331bd335f aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/12c44a8c34f5
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Target Milestone: --- → Firefox 6
Comment 6•13 years ago
|
||
Verified Fixed on: Mozilla /5.0 (Android;Linux armv7l;rv:7.0a1) Gecko/20110607 Firefox/7.0a1 Fennec/7.0a1 Mozilla /5.0 (Android;Linux armv7l;rv:6.0a2) Gecko/20110607 Firefox/6.0a2 Fennec/6.0a2 Device: HTC Desire Z (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•