Closed Bug 1277020 Opened 8 years ago Closed 3 years ago

Full Screen doesn't re-hide notification bar after temporarily showing it

Categories

(Firefox for Android Graveyard :: Audio/Video, defect, P3)

ARM
Android
defect

Tracking

(firefox49 affected, fennec-)

RESOLVED INCOMPLETE
Tracking Status
firefox49 --- affected
fennec - ---

People

(Reporter: justdave, Unassigned)

References

Details

(Whiteboard: [TPE-1])

Firefox Nightly 49.0a1 (2016-05-31)
Motorola Moto X Pure
Android 6.0 (Marshmallow)

When you initially put a video full-screen in Firefox, it does to the full screen (good).  If you then drag your finger down from the top edge of the screen to show the notification bar, the video re-sizes to make room for the bar (good).

In most other apps, when doing this, after 5 seconds or so, the notification bar will disappear again and the video would go back to full-full-screen again.  Firefox doesn't do that. Once the notification bar is visible, it stays visible.  I have to toggle out of full screen and back into it again to get the video to be actually full screen again.
I've noticed this while working on the HLS stuff.
tracking-fennec: --- → ?
Assignee: nobody → gkruglov
tracking-fennec: ? → 51+
Whiteboard: [MobileAS]
Priority: -- → P3
Tried this on current nightly (51) running on my device (moto x play, android 6) - everything works as expected. In fact, video is not resized when notification bar is pulled down - the notification bar is semi-transparent. It automatically slides out of sight after a few seconds. This works both in portrait and landscape mode.

Same behaviour in a couple of emulators (android 6 and 4.4.2) running fennec 51 and 50.

Trying current beta (49), it does indeed have this problem. So it seems that notification bar treatment changed in 50+ (it's semi-transparent so video doesn't need to resize, it slides out, etc). Do we care to investigate trying to uplift some of these changes to 49? It'll be released in less than two weeks now.
Flags: needinfo?(snorp)
Fullscreen behavior was changed by bug 1031519 (I assume that's the behavior you're discussing here), which landed in 50, but only for api level 19+ .  The new behavior isn't available in earlier versions, so I would expect the reporter's behavior to still hold on earlier versions.

You might also take a look at bug 1290882 if considering uplifting 1031519 - the snackbar telling the user to hit the back button to exit fullscreen can be a little confusing now since the back button doesn't appear in the new fullscreen mode until you swipe from the top.
49 is basically cooked, so I think we'll need to let it ship.
Flags: needinfo?(snorp)
Rank: 1
Priority: P3 → P2
Priority: P2 → P1
Priority: P1 → P2
Assignee: gkruglov → nobody
Priority: P2 → P3
Whiteboard: [MobileAS] → [TPE-1]
removing the tracking flag and moving to be tracked under fennec-polish
tracking-fennec: 51+ → -
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.