Remove the `aLibrary` parameter from nsPluginHost::HandleBadPlugin

RESOLVED INVALID

Status

()

Core
Plug-ins
P3
normal
RESOLVED INVALID
13 years ago
5 years ago

People

(Reporter: Biesinger, Unassigned)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

6224   // the |aLibrary| parameter is not needed anymore, after we added |aInstance| which
6225   // can also be used to look up the plugin name, but we cannot get rid of it because
6226   // the |nsIPluginHost| interface is deprecated which in fact means 'frozen'

that interface does not at all seem deprecated, and it doesn't seem to be part of the xpcom plugin api either, so that comment seems wrong. it should probably be removed + that function signature fixed

Updated

5 years ago
Priority: -- → P3
Summary: wrong comment in nsPluginHostImpl → Remove the `aLibrary` parameter from nsPluginHost::HandleBadPlugin

Comment 1

5 years ago
Marking as INVALID as Bug 839731 removed nsPluginHost::HandleBadPlugin entirely.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.