Closed Bug 1480083 Opened Last year Closed Last year

Screen doesn't auto-rotate back to portrait after a fullscreen video is stopped - Android O devices

Categories

(Firefox for Android :: General, defect)

Firefox 63
All
Android
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 64
Tracking Status
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- verified
firefox64 --- verified

People

(Reporter: oana.horvath, Assigned: petru)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Devices: 
Samsung S7 Edge & S8 Note (Android 8.0)
OnePlus 5T (Android 8.0)
Build: Nightly 63 (2018-07-31)

Steps to reproduce:
1. Go to youtube, play a video in fullscreen.
2. Open the task manager and tap the app to return.
* On Nightly, close the PIP and tap the app from the task manager.

Expected result:
When you return to the browser it should rotate to portrait mode.

Actual result:
When the app goes to background, youtube stops the video, but when you return to the browser it's still in landscape mode.

Note:
Can't reproduce on other OS's, like:
Google Pixel (Android P Beta)
Nokia 6 (Android 7.1.1)
OnePlus 2(Android 6.0.1)
Blocks: android-o
No longer depends on: android-o
Hardware: ARM → All
Assignee: nobody → petru.lingurar
Status: NEW → ASSIGNED
Comment on attachment 9006246 [details]
Bug 1480083 - After exiting PIP the restarted activity should have device's orientation; r=sdaswani

:sdaswani only needinfo has approved the revision.
Attachment #9006246 - Flags: review+
Keywords: checkin-needed
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2676a02339f4
After exiting PIP the restarted activity should have device's orientation; r=sdaswani
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/2676a02339f4
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Flags: qe-verify+
Verified as fixed on Nightly 64 (2018-09-07).
Devices:
Devices: 
Samsung S7 Edge & S8 Note (Android 8.0)
OnePlus 5T (Android 8.0)
Flags: qe-verify+
Please request Beta approval on this when you get a chance.
Flags: needinfo?(petru.lingurar)
Comment on attachment 9006246 [details]
Bug 1480083 - After exiting PIP the restarted activity should have device's orientation; r=sdaswani

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1349523
[User impact if declined]: Visual bug - After restarting the app, the screen would be in landscape, irregular of device's actual orientation.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: --
[List of other uplifts needed for the feature/fix]: --
[Is the change risky?]: No
[Why is the change risky/not risky?]: Small change - added code changes orientation to match actual device's orientation when restarting the app.
[String changes made/needed]: --
Flags: needinfo?(petru.lingurar)
Attachment #9006246 - Flags: approval-mozilla-beta?
Comment on attachment 9006246 [details]
Bug 1480083 - After exiting PIP the restarted activity should have device's orientation; r=sdaswani

Uplift approved for 63 beta 5.
Attachment #9006246 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
Verified as fixed on latest Beta build - 63.0b5.
Device: Google Pixel C(Android 8.0.0). Following the steps from the description.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.