Closed Bug 1090576 Opened 10 years ago Closed 3 years ago

Get plugin reloading working properly with content processes

Categories

(Core Graveyard :: Plug-ins, defect, P3)

x86_64
Linux
defect

Tracking

(e10s+)

RESOLVED WONTFIX
Tracking Status
e10s + ---

People

(Reporter: billm, Unassigned)

References

Details

The design we want is probably something like this:
1. If a content process or a plugin process wants to reload plugins, it asks the chrome process to do so.
2. If the chrome process reloads plugins and finds new ones, it broadcasts them to all content processes.
tracking-e10s: --- → ?
Benjamin, can you explain why you think this is a dupe of bug 1096203?
Flags: needinfo?(benjamin)
Plugins bundled as part of addons are registered "late", after Firefox has started up. I concluded that we had already created the content process by that point and therefore the new plugin registration wasn't being sent over properly, which would be fixed by this bug.
Flags: needinfo?(benjamin)
Hi Benjamin, I tried the latest Nightly (39.0a1 2015-03-24), it seems that both the bug 1096203 and bug 1081991 are fixed. Does it mean this bug has been fixed already by some other bug(s) fixing?
Flags: needinfo?(benjamin)
I don't know. Please ask the people who fixed those bugs.
Flags: needinfo?(benjamin)
(In reply to Pin Zhang [:pzhang] from comment #4)
> Hi Benjamin, I tried the latest Nightly (39.0a1 2015-03-24), it seems that
> both the bug 1096203 and bug 1081991 are fixed. Does it mean this bug has
> been fixed already by some other bug(s) fixing?

Hi Bill, should we close this bug?
Flags: needinfo?(wmccloskey)
I don't think we should close this. There's still work to be done afaik. It sounds like your bugs were not related to this bug. I suspect bug 1083281 might have fixed your problem.
Flags: needinfo?(wmccloskey)
billm, is this still relevant?
Flags: needinfo?(wmccloskey)
I think so.
Flags: needinfo?(wmccloskey)
Bill and Benjamin, should we close this? or do something with it?
Flags: needinfo?(wmccloskey)
Flags: needinfo?(benjamin)
This is still behind several user-visible regressions such as bug 1247042.
Flags: needinfo?(benjamin)
Priority: -- → P2
ddurst, this should perhaps be on our plugin-work radar. It's not part of the test pilot project, but it's an e10s regression that could be a problem if there's a 0-day Flash update.
Flags: needinfo?(wmccloskey) → needinfo?(ddurst)
Indeed. Going to block 1282484 because I can't imagine it won't come up. (We probably need a wider meta bug for all this related work.)
Blocks: 1282484
Flags: needinfo?(ddurst)
Blocks: 1326177
Related: test_refresh_navigator_plugins doesn't work with e10s because of this same underlying issue.
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Resolving as wont fix, plugin support deprecated in Firefox 85.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.