PIP toggle displayed on Netflix carousel picker for certain movie clips
Categories
(Web Compatibility :: Tooling & Investigations, defect, P2)
Tracking
(firefox79 wontfix, firefox80 wontfix, firefox81 fixed)
People
(Reporter: cfogel, Unassigned)
References
Details
Attachments
(2 files)
Affected versions
- 80.0b1
Affected platforms
- Windows 10;
Steps to reproduce
- Launch Netflix and login with a valid account;
- Access the Latest or any section;
- Hover over movie tiles to trigger the trailer video;
Expected result
- PIP not displayed on trailer;
Actual result
- PIP displayed;
Regression range
- will check and provide one asap, fwiw it appears to be limited to 80.0b1;
Additional notes
- the report is for tiles appearing in the home-screens not the movie its self;
- attached screenshot with the issue;
- 81.0a1 (2020-07-27) not affected.
Reporter | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Probably not a regression, it looks like we're just hiding it for /browse, not /latest:
Christian, are you able to reproduce this on a /browse page, or only /latest?
Reporter | ||
Comment 2•4 years ago
|
||
Confirmed it on all the main sections (landing, browse, latest, my list).
Updated•4 years ago
|
Comment 3•4 years ago
|
||
(In reply to Cristian Fogel, QA [:cfogel] from comment #2)
Confirmed it on all the main sections (landing, browse, latest, my list).
So this is confusing, it should be blocked on /browse per comment #1. Can you please find a regression window?
Reporter | ||
Comment 4•4 years ago
|
||
Sure, looked in to it and got some additional details:
- it's not limited to Win10, just managed to reproduce on macOS 10.15 as well;
- 78.0.2, 79.0, 81.0a1(2020-08-04) are also affected by this issue, so it's not limited just to the beta branch;
- not a regression;
- visible with addition of pip-toggle button addition @ 2019-april (04-07), mozregression threw some errors before I could get the actual date;
- looking at the account details, in our documents; the account in cause is noted as being in romanian(language-wise) which brings a tought to mind, that it might be related to non en-US accounts.
Comment 5•4 years ago
|
||
Mike, do you know what's going on here? I'm a little bit lost and don't have a netflix account. But I don't understand why we'd display the toggle if we have explicit URL-based code to disable that, unless all of that code is broken, which seems unlikely...
Comment 6•4 years ago
|
||
I suspect this is similar to bug 1653265. I think this is being caused by bug 1605527, where sites that use the history API to "navigate" the user don't have toggle policies re-evaluated.
In the case of this bug, I suspect that cfogel started on a page where the toggle was allowed, and then by clicking on the links to get to browse, etc, the "allow" policy was applied.
I believe by fixing bug 1605527 we can solve this.
Comment 7•4 years ago
|
||
Bug 1605527 is closed, so the weird issue where the toggle appears on all Netflix pages depending on where you enter the application from should be fixed.
I think the way to fix the original bug here is to add /latest
to the policy set. Does this sound good, miketaylr?
Reporter | ||
Comment 8•4 years ago
•
|
||
Since I forgot to add it in the initial comment, after looking at it;
The suggested severity revolves around S2/S3 since it's not respecting the intended specs.
Comment 9•4 years ago
|
||
(In reply to Mike Conley (:mconley) (:⚙️) from comment #7)
Bug 1605527 is closed, so the weird issue where the toggle appears on all Netflix pages depending on where you enter the application from should be fixed.
I think the way to fix the original bug here is to add
/latest
to the policy set. Does this sound good, miketaylr?
Yep, that sounds right to me. We'll fix this in v15.
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Fixed with the in the parent bug (https://bugzilla.mozilla.org/attachment.cgi?id=9175540).
Reporter | ||
Comment 11•4 years ago
|
||
@Thomas, from what I can see 82.0b1, 83.0a1 (2020-09-21) still presents the reported issue; just confirmed it on Windows10/macOS11.0.
Comment 12•4 years ago
|
||
Thanks cfogel, I've filed bug 1666561 about this.
Comment 13•4 years ago
|
||
(In reply to Cristian Fogel, QA [:cfogel] from comment #11)
@Thomas, from what I can see 82.0b1, 83.0a1 (2020-09-21) still presents the reported issue; just confirmed it on Windows10/macOS11.0.
This is unexpected. Can you tell me what preferences you have set under media.videocontrols.picture-in-picture.video-toggle
in about:config? If you set media.videocontrols.picture-in-picture.video-toggle.mode
to -1
, are you still able to reproduce the issue?
Reporter | ||
Comment 14•4 years ago
|
||
Listing all the prefs under that section, this being the case for fresh/new profiles as well:
media.videocontrols.picture-in-picture.video-toggle.always-show -- false
media.videocontrols.picture-in-picture.video-toggle.enabled -- true
media.videocontrols.picture-in-picture.video-toggle.has-used --- false
media.videocontrols.picture-in-picture.video-toggle.min-video-secs -- 45
media.videocontrols.picture-in-picture.video-toggle.mode -- 2
media.videocontrols.picture-in-picture.video-toggle.position -- right
services.sync.prefs.sync.media.videocontrols.picture-in-picture.video-toggle.enabled -- true
Setting media.videocontrols.picture-in-picture.video-toggle.mode
to -1
shows PIP button as well, however it's the old(blue) design.
Comment 15•4 years ago
|
||
(In reply to Cristian Fogel, QA [:cfogel] from comment #14)
Setting
media.videocontrols.picture-in-picture.video-toggle.mode
to-1
shows PIP button as well, however it's the old(blue) design.
What is the URL of the page you're on when you reproduce this?
Reporter | ||
Comment 16•4 years ago
•
|
||
Button shows for some videos in these sections: shows, movies, my page.
[Main-page] and [latest] section appear to work just fine and not display the PIP button.
Updated•4 years ago
|
Description
•