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)
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.
Updated•10 years ago
|
tracking-e10s:
--- → ?
Reporter | ||
Comment 2•10 years ago
|
||
Benjamin, can you explain why you think this is a dupe of bug 1096203?
Flags: needinfo?(benjamin)
Comment 3•10 years ago
|
||
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)
Comment 4•10 years ago
|
||
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)
Comment 5•10 years ago
|
||
I don't know. Please ask the people who fixed those bugs.
Flags: needinfo?(benjamin)
Comment 6•10 years ago
|
||
(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)
Reporter | ||
Comment 8•10 years ago
|
||
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)
Comment 9•10 years ago
|
||
Thanks.
Comment 12•9 years ago
|
||
Bill and Benjamin, should we close this? or do something with it?
Flags: needinfo?(wmccloskey)
Flags: needinfo?(benjamin)
Comment 13•9 years ago
|
||
This is still behind several user-visible regressions such as bug 1247042.
Flags: needinfo?(benjamin)
Updated•9 years ago
|
Priority: -- → P2
Comment 14•8 years ago
|
||
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)
Comment 15•8 years ago
|
||
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)
Comment 17•8 years ago
|
||
Related: test_refresh_navigator_plugins doesn't work with e10s because of this same underlying issue.
Comment 18•6 years ago
|
||
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
Comment 19•6 years ago
|
||
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
Comment 20•3 years ago
|
||
Resolving as wont fix, plugin support deprecated in Firefox 85.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•