224349 - Top Menu Bar Missing

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Jeff Grossman]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.6a) Gecko/20031031 Firebird/0.7+/Built by JGrossman
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.6a) Gecko/20031031 Firebird/0.7+/Built by JGrossman

On the build which I built on 10/31/03, the top menu bar where File, Edit,
Bookmarks, Tools, etc. does not display on OS X.  I am running Panther, and this
is the first build which showed this error.  I only have the blue apple and
Mozilla Firebird on the top titlebar.  I do not have the File, Edit, Bookmarks,
and Tools menu options.

Reproducible: Always

Steps to Reproduce:
1.  Open up Firebird from 10/31/03.
2.  Notice the missing menus.
3.

Comment 1

[Hiro]

Confirmed.

OSX 10.3 
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; ja-JP; rv:1.6a) Gecko/20031028
Firebird/0.7+

Comment 2

[Jeff Grossman]

I also noticed today that it does not even have the About MozillaFirebird option at the top of the 
only menu which does appear.

Comment 3

[cls]

My debug build starts up with:
No Persistent Registry Found.
Type Manifest File:
/mnt/cls/src/moz/main/obj-dbg/dist/MozillaDebug.app/Contents/MacOS/components/xpti.dat
WARNING: Could not load XPT Zip loader, file
../../../../../mozilla/xpcom/reflect/xptinfo/src/xptiInterfaceInfoManager.cpp,
line 1168
WARNING: Could not load XPT Zip loader, file
../../../../../mozilla/xpcom/reflect/xptinfo/src/xptiInterfaceInfoManager.cpp,
line 1168
nsNativeComponentLoader: autoregistering begins.


The 1.6a release does not exhibit this problem.  Bouncing to xpcom.

Comment 4

[Doug Turner (:dougt)]

fix for 223084?

Comment 5

[Benjamin Smedberg]

yes, and I fixed it with a supplementary checkin on 11/1... we need to pass a
null "aFile" to the static component loader during autoregistration so that we
don't repeatedly autoregister the static components.

--- nsComponentManager.cpp	8 Sep 2003 00:31:17 -0000	1.241
+++ nsComponentManager.cpp	1 Nov 2003 14:46:08 -0000	1.242
@@ -3197,9 +3197,9 @@ nsComponentManagerImpl::AutoRegisterImpl
     rv = mNativeComponentLoader->AutoRegisterComponents((PRInt32)when, dir);
     if (NS_FAILED(rv)) return rv;
 
 #ifdef ENABLE_STATIC_COMPONENT_LOADER
-    rv = mStaticComponentLoader->AutoRegisterComponents((PRInt32)when, dir);
+    rv = mStaticComponentLoader->AutoRegisterComponents((PRInt32)when, inDirSpec);
     if (NS_FAILED(rv)) return rv;
 #endif

Comment 6

[cls]

This doesn't appear to be the same issue.  I just updated my tree and the Top
Menu bar still only contains a single item.   Compiling with
--disable-static-component-loader didn't make a difference either.

Comment 7

[Jeff Grossman]

Yes, this is still not fixed.  I just compiled a build this morning from CVS and
it still only shows Mozilla Firebird at the top of the screen.

Comment 8

[Benjamin Smedberg]

Do either of you have a debug build in this state? could you delete compreg/xpti
and use NSPR logging to get a log and attach it here?:

export NSPR_LOG_MODULES=nsComponentManager:4
export NSPR_LOG_FILE=/path/to-log-file

Comment 9

[cls]

Attachment: nspr log nsComponentManager:4

Comment 10

[cls]

Btw, I did try building with flat chrome and the problem still persists so it
may not be zip related at all.

Comment 11

[Benjamin Smedberg]

Oops, I had thought these were static builds. But that doesn't make it any
easier, but I think we can rule out the static component loader itself. Hrm...

It looks like
http://lxr.mozilla.org/mozilla/source/xpcom/reflect/xptinfo/src/xptiInterfaceInfoManager.cpp#1150
is trying to create the xptziploader before it has been registered.

Can you break into a debugger at
http://lxr.mozilla.org/mozilla/source/xpcom/reflect/xptinfo/src/xptiInterfaceInfoManager.cpp#1168
and pull me a stacktrace? right now I am guessing that the new "suspend timers
on sleep" code is causing the observer service to startup early, before things
are registered successfully. Which is quite a pickle.

--BDS

Comment 12

[cls]

Fixed by checkin for bug 224526

Comment 13

[Jesse Ruderman]

*** Bug 224527 has been marked as a duplicate of this bug. ***