Closed
Bug 670079
Opened 13 years ago
Closed 13 years ago
stop caching plugin instances
Categories
(Core Graveyard :: Plug-ins, defect)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla8
People
(Reporter: jaas, Assigned: jaas)
References
Details
Attachments
(1 file)
16.52 KB,
patch
|
jst
:
review+
|
Details | Diff | Splinter Review |
I don't think we ever want to restart a plugin instance so we can stop caching them entirely and remove code that looks for an old stopped instance to re-use when instantiating. I'm not sure we want to support NPPVpluginKeepLibraryInMemory either. Right now that marks an instance for caching, which effectively prevents its plugin from ever unloading since an instance will remain indefinitely. This is a silly way to implement it, a better way would be to just mark the plugin as do-not-unload when this is set on any instance. However, since we don't unload plugins by default anyway this isn't really an issue.
Attachment #544733 -
Flags: review?(jst)
Comment 2•13 years ago
|
||
Comment on attachment 544733 [details] [diff] [review] fix v1.0 Yeah, this never seemed like a good idea to me either. r=jst
Attachment #544733 -
Flags: review?(jst) → review+
pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/b4725ef65601
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 4•13 years ago
|
||
Does this fix bu 659348?
Comment 5•13 years ago
|
||
If I'm reading https://wiki.mozilla.org/RapidRelease#Merge_and_Release_Dates correctly this fix was first in Firefox 8.
Target Milestone: --- → mozilla8
Comment 6•13 years ago
|
||
Yes. And this is confirmed by the names of the 2011-07-08 and 2011-07-09 mozilla-central nightlies, which claim to be "8.0a1".
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•