Videos aren't auto rotated on mobile depending on screen size
Categories
(Firefox for Android Graveyard :: Audio/Video, defect, P3)
Tracking
(Not tracked)
People
(Reporter: csheany, Unassigned)
References
Details
User Agent: Mozilla/5.0 (Android 7.1.1; Tablet; rv:66.0) Gecko/66.0 Firefox/66.0
Steps to reproduce:
- Open mozilla.org/en-us/firefox
- Play a video
- Enter fullscreen
Actual results:
The video is in portrait.
Expected results:
The video should be in landscape.
Comment 1•6 years ago
|
||
I was able to reproduce the issue.
Tested on the latest Nightly 66.0a1 (2019-01-16) on Sony Xperia Z2 (Android 6.0.1).
I will confirm the issue.
Updated•6 years ago
|
Comment 2•6 years ago
|
||
Ran a regression:
Last good revision: 654c9274673479fc772eef2f8a3002982e931a62
First bad revision: 895cc0297ce2e47092dc5736cd444a573ba03ed2
Comment 3•6 years ago
|
||
Hi, from what I see, it seems that this issue is caused by the same bug as bug 1520455. Please consider that in your investigation, thanks.
Hiro, can you take a look?
Comment 5•6 years ago
|
||
Sure.
On my environments some of videos on mozilla.org/en-us/firefox has been originally played in portrait on fullscreen. So I don't know what the problem is actually, but I believe bug 1520455 also fixed this issue.
csheany, can you still see the issue on nightly?
I am still seeing the issue.
It seems to depend on screen size.
I was going to edit the summary.
To confirm, are you playing the video first?
Comment 7•6 years ago
|
||
Yeah, I did play the video then pushed the fullscreen icon. Anyway, that's odd bug 1520455 made bug 1423013 (which regressed this) completely off on fullscreen state. The latest nightly hasn't included the fix for bug 1520455 yet? Let me check it.
Comment 8•6 years ago
|
||
I've confirmed the latest nightly (rev: f0c23db0d035) has included bug 1520455. It seems that the issue happens only on Tablet.
Comment 9•6 years ago
|
||
CCing timdream, we might need to tweak toolkit side due to bug 1423013. Given that since bug 1520455 we no longer use the minimum scale size on fullscreen state but the problem still persists, that mean that the orientation change depends on the layout viewport size before fullscreen? Just a guess.
Comment 10•6 years ago
|
||
csheany, what is your device screen size? I can't yet reproduce on an Android emulator that the screen size is 1280x800.
Reporter | ||
Comment 11•6 years ago
|
||
The tablet is 1024x768 however I notice it when the window is 512x384.
Comment 12•6 years ago
|
||
No luck on 1024x768 or on 512x384 either. I've been trying both of desktop and mobile versions of the site.
miralobontiu, would you mind helping me to reproduce this issue on Android emulators? Oh I just noticed that Xperia Z2 isn't table at all. I have a similar device, Z3 Compact, so it should be reproducible on my phone.
Reporter | ||
Comment 13•6 years ago
|
||
1024x768 isn't affected.
The other dimensions should have been 768x512.
Comment 14•6 years ago
|
||
Thanks, I will try 768x512. That's said, I started suspecting the regression range was misidentified due to bug 1423013.
IIUC, the orientation change happens based on the video element's videoHeight and videoWidth, so it shouldn't be affected by the minimum scale change.
Comment 15•6 years ago
|
||
Can't reproduce it on 768x512 either.
Comment 16•6 years ago
|
||
Hey csheany, can you please query the videoHeight and videoWidth values? It should be; videoHeight:1080 and videoWidth:1920 for the first video on the site.
You can use about:debugging and connect with the device and run document.fullscreenElement.videoHeight or document.fullscreenElement.videoWidth on fullscreen state.
Reporter | ||
Comment 17•6 years ago
|
||
I appreciate your efforts.
Unfortunately that is beyond my capacity.
Would the page source provide that information?
Comment 18•6 years ago
|
||
Never mind, then. Also the place I linked in comment 14 is used only if media.videocontrols.lock-video-orientation is true, so it's not related at all, I suppose.
Comment 19•6 years ago
|
||
Hi,
I wasn`t able to reproduce the issue anymore on Sony Xperia Z2 (Android 6.0.1), a tablet which has 1080 x 1920 pixels, 16:9 ratio (~424 ppi pixel density).
Tested on the latest Nightly 66.0a1 (2019-01-23), on Beta 65.0b13 and RC 65.0.
I will remove the NI.
Thanks!
Comment 20•6 years ago
|
||
miralobontiu (or Stefan), would you mind finding which commit fixed this too? I think the issue what you guys saw was fixed by bug 1520455, and I guess the problem csheany reported is originally different from the one. Thanks!
Reporter | ||
Comment 21•6 years ago
|
||
I believe it is the same issue.
It wouldn't work at all without the pref you mentioned in Comment 18.
Comment 22•6 years ago
|
||
I am confused. Can you please clarify steps to reproduce? The problem happens only if media.videocontrols.lock-video-orientation is false, right? Then I can see the issue on Beta and Release on my phone (with portrait mode).
Comment 23•6 years ago
|
||
Also it would be nice to confirm whether your issue regressed by bug 1423013.
Reporter | ||
Comment 24•6 years ago
|
||
The STR haven't changed.
If the pref is false video doesn't auto rotate.
I'm not sure if it was caused by Bug 1423013.
Comment 25•6 years ago
|
||
Great. I am pretty sure that the issue you are seeing is not a regression caused by bug 1423013. (Probably it's not a regression, I guess) Unassigning myself. Also I am degrading the priority. (Please feel free to re-upgrade if someone thinks this should be high priority)
Comment 26•6 years ago
|
||
Probably we won't fix this issue on Firefox 64 and 65 either.
Reporter | ||
Comment 27•6 years ago
|
||
If the pref is false, it is not expected to.
Comment 28•6 years ago
|
||
Are you saying the the issue happens if the pref value is true? Just to confirm the issue happens on Firefox 64?
Comment 29•6 years ago
|
||
Also please note the conditions when the problem happens. The pref value and the orientation of the device (I supposed you are using the device as portrait mode).
Reporter | ||
Comment 30•6 years ago
|
||
Correct.
What are you seeing on 64?
Comment 31•6 years ago
|
||
to confirm, the 'correct' means that you are seeing the issue on Firefox 64 as well and the pref value is true, right?
I can see the issue only if the pref is false on Firefox 64. So that means I can't reproduce your issue at all.
Reporter | ||
Comment 32•6 years ago
|
||
In my case, with multi-window the device is in "landscape" and Firefox is in "portrait" and the pref is true.
I took that to be reprentative of mobile/phone which from Mira and Stefan's comments was accurate to a degree.
Comment 33•6 years ago
|
||
multi-window! I've never tried it. :/ Probably it's the key factor.
Comment 34•6 years ago
|
||
We should reset the bug status.
Reporter | ||
Comment 35•6 years ago
|
||
Correct that the pref value is true.
I have only tested Nightly.
Comment 36•6 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #20)
miralobontiu (or Stefan), would you mind finding which commit fixed this too? I think the issue what you guys saw was fixed by bug 1520455, and I guess the problem csheany reported is originally different from the one. Thanks!
Hi guys!
I found which commit fixed this issue: https://hg.mozilla.org/mozilla-central/log/f0c23db0d035dbe81e23eb4d619e493e38582d24 - it was the one from 22nd of January 2019.
I tested with Sony Xperia Z2 (Android 6.0.1).
If I can help you with other information, please let me know.
Thank you!
Updated•6 years ago
|
Updated•5 years ago
|
Reporter | ||
Comment 37•5 years ago
|
||
If there was an issue in a normal environment and can no longer be reproduced that...
Updated•4 years ago
|
Description
•