Closed
Bug 1404713
Opened 7 years ago
Closed 5 years ago
Can't enter fullscreen on youtube after moving a tab to another window.
Categories
(Firefox :: Tabbed Browser, defect, P2)
Tracking
()
RESOLVED
WORKSFORME
| Tracking | Status | |
|---|---|---|
| firefox-esr52 | --- | unaffected |
| firefox56 | --- | wontfix |
| firefox57 | + | wontfix |
| firefox58 | + | wontfix |
| firefox59 | --- | wontfix |
| firefox60 | --- | fix-optional |
| firefox61 | --- | fix-optional |
| firefox72 | --- | unaffected |
| firefox73 | --- | unaffected |
| firefox74 | --- | unaffected |
People
(Reporter: johan.charlez, Unassigned)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
(deleted),
text/plain
|
Details |
STR:
1. Open two window.
2. Open a video on youtube in one window.
3. Move the "youtube"-tab to the other window.
4. Double-click or click the fullscreen button on the video.
Expected: Video goes fullscreen.
Actual: Nothing.
Note:
If you blur and return focus to the window with the broken "Youtube"-tab fullscreen magically fixes itself.
I've attached the output from the Browser Console, the console was cleared immediately before moving the tab between the two windows.
This is a regression, currently working on narrowing it down with mozregression, details incoming.
Keywords: regression
From mozregression-gui's "Bisection informations":
app_name: firefox
build_date: 2017-05-23 18:51:36.638000
build_file: C:\Users\[redacted]\.mozilla\mozregression\persist\aa48bb3f4944--autoland--firefox-55.0a1.en-US.win64.zip
build_type: inbound
build_url: https://queue.taskcluster.net/v1/task/N0XM82LgT8ysRFUytWqWeQ/runs/0/artifacts/public%2Fbuild%2Ffirefox-55.0a1.en-US.win64.zip
changeset: aa48bb3f494410de514041e8c06ca15c38680181
pushlog_url: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=d11aec13b38e0837c07d0906c1ab506e162b2a2f&tochange=aa48bb3f494410de514041e8c06ca15c38680181
repo_name: autoland
repo_url: https://hg.mozilla.org/integration/autoland
task_id: N0XM82LgT8ysRFUytWqWeQ
I'm not entirely sure how to interpret this myself, but to me it looks like bug 1362866 regressed this?
Mike, it looks like you worked on the bug mentioned, do you think it could be the cause of this regression? (sorry if this is misuse of ni? by the way.)
Flags: needinfo?(mconley)
|
||
Comment 3•7 years ago
|
||
Confirmed that this is bug 1362866...
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=6d4744a6c81c498cc81cad11b4ead64660a3d269&tochange=aa48bb3f494410de514041e8c06ca15c38680181
[Tracking Requested - why for this release]:
While the symptoms here aren't terrible, and we've shipped this in 55 and 56 now, I am... concerned... that the media fullscreen code doesn't think the tab has focus after moving it to a new window. It should know that it does, and the internal state being bust might have other ramifications. :-(
Blocks: 1362866
status-firefox56:
--- → wontfix
status-firefox57:
--- → affected
status-firefox58:
--- → affected
status-firefox-esr52:
--- → unaffected
tracking-firefox57:
--- → ?
|
|
||
Comment 4•7 years ago
|
||
(FWIW, the relevant bit of the console output is "Request for fullscreen was denied because requesting element is not in the currently focused tab." )
|
|
||
Comment 5•7 years ago
|
||
We must be failing this check:
https://dxr.mozilla.org/mozilla-central/rev/11fe0a2895aab26c57bcfe61b3041d7837e954cd/dom/base/nsDocument.cpp#11927-11930
if (!nsContentUtils::IsChromeDoc(this) && !IsInActiveTab(this)) {
DispatchFullscreenError("FullscreenDeniedNotFocusedTab");
return false;
}
IsInActiveTab looks like this:
https://dxr.mozilla.org/mozilla-central/rev/11fe0a2895aab26c57bcfe61b3041d7837e954cd/dom/base/nsDocument.cpp#11786-11823
and checks both the docshell's isActive flag and whether the focus manager thinks the docshell's root window is the active window.
Too late for uplift to 57. We could still take a patch for 58 though.
tracking-firefox58:
--- → +
|
||
Updated•7 years ago
|
Priority: -- → P2
|
||
Updated•7 years ago
|
|
||
Updated•7 years ago
|
status-firefox59:
--- → wontfix
status-firefox60:
--- → fix-optional
status-firefox61:
--- → fix-optional
|
||
Comment 8•5 years ago
|
||
Apologies - declaring needinfo bankruptcy on needinfo's greater than 2 years old.
Flags: needinfo?(mconley)
|
||
Comment 9•5 years ago
|
||
Hi,
I'm not able to reproduce this issue on Windows 10 for Firefox version Nightly 74.0a1 (2020-01-23) (64-bit) - Beta 73.0b8 (64-bit) - Release 72.0.2 (64-bit). Marking those flags as unaffected.
status-firefox72:
--- → unaffected
status-firefox73:
--- → unaffected
status-firefox74:
--- → unaffected
|
|
||
Comment 10•5 years ago
|
||
Reporter, are you still seeing this? It's been a while, so if this is fixed it'd be useful to close the bug; if not, it'd be useful to have more details as to how to reproduce, so we can clarify which versions remain affected...
Flags: needinfo?(johan.charlez)
|
Reporter | |
Comment 11•5 years ago
|
||
(In reply to :Gijs (he/him) from comment #10)
Reporter, are you still seeing this? It's been a while, so if this is fixed it'd be useful to close the bug; if not, it'd be useful to have more details as to how to reproduce, so we can clarify which versions remain affected...
Sorry for the delayed reply.
I cannot reproduce this issue with Firefox 72.0.2 (64-bit).
Flags: needinfo?(johan.charlez)
|
|
||
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
|
|
||
Comment 12•5 years ago
|
||
Thanks for following up Johan!
You need to log in
before you can comment on or make changes to this bug.
Description
•