Bug 15217 – Plugin complains that xpcom_core.dll missing

• Component: Plug-ins
• Status: RESOLVED FIXED

[Anantha Keesara]

Steps:
1. load http://tv.fenbei.com/ to see the following error.

---------------------------
Loading “FenBei TV 在线电影台”: Safari.exe - Unable To Locate Component
---------------------------
This application has failed to start because xpcom_core.dll was not found. Re-installing the application may fix this problem. 
---------------------------
OK   
---------------------------

[Mark Rowe (bdash)]

On the Mac I see a sheet saying: "The page “FenBei TV 在线电影台” has content of MIME type “application/x-oleobject”. Because you don’t have a plug-in installed for this MIME type, this content can’t be displayed."

As far as I can tell this has nothing to do with Flash.  It sounds like you may have a Firefox plugin installed that attempts to handle the ActiveX MIME types.

[Matt Lilek]

I just get a broken plugin icon with Safari 3.0.3 and Safari uses no CPU.

[Mark Rowe (bdash)]

Anantha, can you please check which plugins are installed?  Under the Help menu, choose Installed Plugins.  Look through the list to find which plugin is claiming "application/x-oleobject".  I suspect that plugin will be a Firefox-specific plugin that was written to allow Firefox to support ActiveX.  Please report back with what you find.

[Anantha Keesara]

This is what I see:

Mozilla ActiveX control and plugin support
Mozilla ActiveX control and plugin module — from file “npmozax.dll”.


MIME Type	                Description	Extensions
application/x-oleobject	          ActiveX	*.ocx
application/oleobject	          ActiveX	*.ocx

[Alexey Proskuryakov]

This plugin clearly requires Mozilla/Firefox, and cannot work with Safari.

Perhaps we should skip it when loading plug-ins to avoid getting such errors?

[David Kilzer (:ddkilzer)]

<rdar://problem/5488870>

[Alexey Proskuryakov]

We now have PluginDatabaseWin::isPluginBlacklisted() just for this.

[Alexey Proskuryakov]

Created attachment 16897 [details]
proposed fix

I haven't verified that this patch fixes the problem, since Windows Safari beta is still incompatible with TOT, but it seems simple enough.

[Adam Roben (:aroben)]

Comment on attachment 16897 [details]
proposed fix

r=me, though I think the comment could go beneath the if instead of next to it.

[Alexey Proskuryakov]

Committed revision 27175.

Someone with access to a working Safari build will need to verify that this actually helps.

[Alexey Proskuryakov]

I've tested the fix now, and it does work. Of course, there is still an alert about missing plugin, and no video is displayed, but that's expected behavior.