Closed Bug 44810 Opened 24 years ago Closed 24 years ago

Mac hangs with a plugin in the Plugins folder

Categories

(Core Graveyard :: Plug-ins, defect, P3)

PowerPC
Mac System 9.x
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: peter.vanderbeken, Assigned: serhunt)

Details

From Bugzilla Helper:
User-Agent: Mozilla/4.07 (Macintosh; I; PPC, Nav)
BuildID:    2000061408

The latest check-in (version 1.130) by av@netscape.com into mozilla/modules/
plugin/nglsrc/nsPluginHostImpl.cpp breaks plug-ins on Mac. Line 2478 does not 
work for Mac:

nsresult rvIsLayoutPath = compManager->SpecForRegistryLocation("rel:gkhtml.dll", 
getter_AddRefs(path));

gkhtml.dll is not a library on Mac.

Reproducible: Always
Steps to Reproduce:
Make sure a plugin (for example MRJPlugin) is in the Plugins folder. Launch with 
Profile Manager, then try to create a new profile.

Actual Results:  I see the following in the console:

nsNativeComponentLoader: 
GetFactory(HD:Mozilla:mozilla:dist:viewer_debug:Components:gkhtml.dll) Load 
Failed with error: error 0.
Cut and paste thing, sorry, I'll remove it. But it should not break any existing 
functionality, it should just fail and go like it was before. What do you mean 
by 'Plug-ins broken on Mac'?
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
I'll change it. When i have a plugin in the Plugins folder, mozilla hangs.

First i get:

nsNativeComponentLoader: 
GetFactory(HD:Mozilla:mozilla:dist:viewer_debug:Components:gkhtml.dll) Load 
Failed with error: error 0.

Then later these two:

###!!! ASSERTION: NS_ENSURE_TRUE(docShellElement) failed: 'docShellElement', 
file nsXULWindow.cpp, line 733

and

###!!! ASSERTION: NS_ENSURE_TRUE(window) failed: 'window', file 
nsContentTreeOwner.cpp, line 567

and after a couple of these it hangs. Removing the plugin resolves the problem.
Summary: Plug-ins broken on Mac → Mac hangs with a plugin in the Plugins folder
This is a bad bug (I'm being asserted to death); can we get a fix soon?
Severity: normal → critical
Keywords: nsbeta2
I checked in the fix for LAYOUT_DLL yesterday. Do you still see this?
OK, as brade suggested I changed rel:LAYOUT_DLL to LAYOUT_DLL for Mac which 
seems to fix the problem. Peter would you please confirm?
Fixed. Thanks.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
marking verified (2000071708)
Status: RESOLVED → VERIFIED
OS: Mac System 9.x
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.