227653 - Install.refreshPlugins should autoregister XPCOM components

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Installer: XPInstall Engine, defect)

Description

[Ilya Konstantinov]

Install.refreshPlugins method should autoregister XPCOM components before
refreshing plugin info, just like PluginArray.refresh (navigator.plugins.refresh
from JS) does.

You can see the PluginArray.refresh implementation at:
http://lxr.mozilla.org/mozilla/dom/src/base/nsPluginArray.cpp#183
and our current implementation in XPInstall at:
http://lxr.mozilla.org/mozilla/xpinstall/src/nsXPIProxy.cpp#47

Basically, we're missing the call to AutoRegister() on the XPCOM component manager.

Fixing this bug would allow us to XPInstall XPCOM components and make them
available right away, without requiring a restart, by simply calling
RefreshPlugins after installation.

(And have I mentioned I made it trivial to fix? :)

Comment 1

[Ilya Konstantinov]

This bug is only relevant for Seamonkey now, as FF/TB installations are done without JS code now. What should I assign it to?

Comment 2

[Benjamin Smedberg]

This code is dead.