Closed Bug 702907 Opened 8 years ago Closed 8 years ago

Zooming should be disabled while a video is in fullscreen mode

Categories

(Firefox for Android :: General, defect, P3)

ARM
Android
defect

Tracking

()

VERIFIED FIXED
Firefox 13
Tracking Status
firefox15 --- verified
blocking-fennec1.0 --- +

People

(Reporter: xti, Assigned: blassey)

References

Details

Attachments

(1 file)

Mozilla/5.0 (Android;Linux armv7l;rv:11.0a1)Gecko/20111115
Firefox/11.0a1 Fennec/11.0a1
Devices: Motorola Droid 2
OS: Android 2.3.3

Steps to reproduce:
1. Open Fennec App
2. Browse to http://random.pavlov.net/video.html
3. Long tap on video for the Context Menu
4. Tap on Full Screen option
5. Perform a zoom-in/out action

Expected result:
Zooming is disabled while a video is in fullscreen mode.

Actual result:
Zooming still works even if the video is playing in fullscreen mode.
OS: Windows 7 → Android
Hardware: x86 → ARM
Priority: -- → P4
QA verifiers : Please test bug Bug 706694 if this gets resolved.  that bug involves panning not zooming.
(In reply to Naoki Hirata :nhirata from comment #1)
> QA verifiers : Please test bug Bug 706694 if this gets resolved.  that bug
> involves panning not zooming.

I've commented in bug 706694. It doesn't involve panning either. It seems that all content is automatically zoomed out if Full Screen option is tapped.
Priority: P4 → P3
blocking-fennec1.0: --- → +
Attached patch patchSplinter Review
Assignee: nobody → blassey.bugs
Attachment #604071 - Flags: review?(mbrubeck)
Comment on attachment 604071 [details] [diff] [review]
patch

I'm slightly worried this is too heavy-handed (are there other use cases of requestFullScreen where zooming *should* remain enabled?) but it seems like a reasonable step.  If it causes problems for other use cases we can fix them.
Attachment #604071 - Flags: review?(mbrubeck) → review+
https://hg.mozilla.org/mozilla-central/rev/fbace64ae2eb
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 13
Zoom is disabled in Full Screen mode. Closing bug as verified fixed on:

Firefox 15.0a1 (2012-05-21)
Device: Galaxy Nexus
OS: Android 4.0.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.