Closed Bug 1231306 Opened 7 years ago Closed 7 years ago

[e10s] Remote process doesn't immediately change Flash plugin state when I set it to "Always Activate"


(Core Graveyard :: Plug-ins, defect)

Not set


(e10sm8+, firefox45 fixed, firefox46 fixed)

Tracking Status
e10s m8+ ---
firefox45 --- fixed
firefox46 --- fixed


(Reporter: arni2033, Assigned: billm)





(1 file)

>>>   My Info:   Win7_64, Nightly 45, 32bit, ID 20151206030448
1. Open "about:addons", set Shockwave Flash to "Never Activate"
2. Restart the browser
3. Open in a new tab
4. Open "about:addons" (or switch to opened tab), set Flash to "Always Activate"
5. Open (flash) in a new tab

 There's a prompt window "What should Nighly do with this file?"

 The flash player should load.

 To kill the remote process (or close all remote tabs), then open flash player in a new tab.
 Note that if dom.ipc.processCount is set to > 1, then you can only hope that when you kill a process,
 the new tab will be opened in a new process.
Flags: needinfo?(jmathies)
Sounds like an e10s blocker. Removing my ni so this falls into our triage list.
tracking-e10s: ? → ---
Flags: needinfo?(jmathies)
tracking-e10s: --- → ?
Assignee: nobody → wmccloskey
I can reproduce this.
Attached patch patchSplinter Review
There were some bookkeeping issues here. The content process frequently sends up queries asking for the current list of plugins. Both the parent and child record an "epoch" value, which is an integer that changes whenever the list of plugins changes. The parent will not send down a new list of plugins if the child's epoch value is the same as the parent's--in this case, the two lists are already synchronized.

One problem here is that we weren't updating the epoch when changing the enabled state of a plugin. Another problem is that enabling a plugin should cause us to register it with the category manager so that it can be used as a "full page plugin" (typically only used for Acrobat or if you navigate directly to an .flv file). The child process was not doing the category manager registration.
Attachment #8701659 - Flags: review?(jmathies)
Attachment #8701659 - Flags: review?(jmathies) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Has STR: --- → yes
Blocks: 1190364
Duplicate of this bug: 1190364
Comment on attachment 8701659 [details] [diff] [review]

We'd like to land this for the beta 45 experiment. It's been in for a while and is low risk.

Approval Request Comment
[Feature/regressing bug #]: e10s
[User impact if declined]: fullscreen plugins may not work
[Describe test coverage new/current, TreeHerder]: on m-c
[Risks and why]: low
[String/UUID change made/needed]: none
Attachment #8701659 - Flags: approval-mozilla-aurora?
Comment on attachment 8701659 [details] [diff] [review]

Taking it for the experiment
Attachment #8701659 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.