Call TabChild::MakeHidden and TabChild::MakeVisible for all oop-iframes when switching tabs
Categories
(Core :: Layout, enhancement, P3)
Tracking
()
People
(Reporter: jwatt, Assigned: rhunt)
References
(Blocks 1 open bug)
Details
The root presContext is told to change the state of any plugins to “hidden” whenever a Tab is hidden (by TabChild::MakeHidden). Notifying the root presContext of the top-level document won't be enough in a Fission world. We'll need to notify all oop-frames too.
Comment 1•6 years ago
|
||
"In early 2020, Flash support will be completely removed from consumer versions of Firefox. The Firefox Extended Support Release (ESR) will continue to support for Flash until the end of 2020."
https://developer.mozilla.org/en-US/docs/Plugins/Roadmap
So it seems to me that making the "PluginGeometryUpdates" parts of
TabChild::MakeHidden work in a Fission world is very low priority.
I expect all that code to be removed once we stop supporting plugins.
(Please correct me if I'm missing something.)
Comment 2•6 years ago
|
||
So I wonder if we could get away with simply not supporting Flash
in OOP <iframes> at all?
Reporter | ||
Comment 3•6 years ago
|
||
Maybe.
Just to be clear, this bug is about nsRootPresContext::ComputePluginGeometryUpdates.
Reporter | ||
Comment 4•6 years ago
|
||
needinfo is just to make sure you're aware of this bug in case it duplicates anything gfx is working on.
Reporter | ||
Comment 5•6 years ago
|
||
Neha, do you know the answer to comment 2?
Comment 6•6 years ago
|
||
(In reply to Mats Palmgren (:mats) from comment #1)
"In early 2020, Flash support will be completely removed from consumer versions of Firefox. The Firefox Extended Support Release (ESR) will continue to support for Flash until the end of 2020."
https://developer.mozilla.org/en-US/docs/Plugins/RoadmapSo it seems to me that making the "PluginGeometryUpdates" parts of
TabChild::MakeHidden work in a Fission world is very low priority.
I expect all that code to be removed once we stop supporting plugins.
(Please correct me if I'm missing something.)
Considering flash to not be supported starting early 2020, I think it is a safe bet to not support Flash for OOP iframes.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 7•6 years ago
|
||
I'm actually just going to duplicate this with bug 1525720 and do the work there.
Description
•