bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.

PT Bookmarks menu stops working after installing an XPI add on

RESOLVED FIXED in mozilla1.1alpha



17 years ago
7 years ago


(Reporter: Jason Kersey, Assigned: janv)



Windows ME

Firefox Tracking Flags

(Not tracked)



(1 attachment, 1 obsolete attachment)



17 years ago
Open the browser, and click the PT bookmarks button.  I get the top three
hardcoded menuitems, add, file, manage, and that's it.  Is this from the
tabbrowser checkin?  The menubar menu works.
wfm with win2k and a 2h old CVS build...
BTW:Stupid bugzilla newbie :Please include in all your bug reports the build ID. :-)

Comment 2

17 years ago
I saw this too, but it went away. It was in the 20010920 or 19 build. Now in
20010921 I can't get it to happen again. 

I was on Win98

Comment 3

17 years ago
Button works fine for me in current builds.
Last Resolved: 17 years ago
Resolution: --- → WORKSFORME

Comment 4

17 years ago
I see this in 2001092208.  Existing profile.  blake said this may be hewitt.  to 
him to look at.  I get:

Error: this.docShell has no properties
Source File: 
chrome://global/content/bindings/browser.xml#browser.webNavigation (getter)
Line: 0

in the js console, unsure if this is related.
Resolution: WORKSFORME → ---

Comment 5

17 years ago
Assignee: hyatt → hewitt

Comment 6

17 years ago
I'm with Mike, I saw this but then it went away (i think with a few restarts,
not with a new build).

Comment 7

17 years ago
I'm seeing this again (right now) build 2001092103

I've closed and reopened, restarted the computer, mozilla and opened new windows.
No Luck. Still can't get it to work.

attaching screenshot.

Comment 8

17 years ago
Created attachment 50501 [details]

Comment 9

17 years ago
see also bug 101239, where the menu popup won't rollup after first shown.

Comment 10

17 years ago
works for me
Last Resolved: 17 years ago17 years ago
Resolution: --- → WORKSFORME

Comment 11

17 years ago
come to my cube and see it.  it still doesn't work in a new profile.
Resolution: WORKSFORME → ---

Comment 12

17 years ago
oh, wait, I misunderstood the problem... I do indeed see it now.
Priority: -- → P1
Target Milestone: --- → mozilla0.9.5


17 years ago
Blocks: 101793

Comment 13

17 years ago
I don't know how or why, but I think I can reproduce this by installing an XPI.
I realized that this went away with new builds and then resurfaced sometimes
after I had been working, and I believe its when I install something in Mozilla.
I've installed the Preferences toolbar and Mozgest, which all seem to cause this
problem. You can get the Preferences toolbar at
http://www.xulplanet.com/downloads/view.cgi?category=applications&view=all and
try to see whether you reproduce it.

Comment 14

17 years ago
Yes!! I did notice this too awhile back but until now had forgotten it.  I have
the preferences toolbar installed too - not sure if that was what had done it
but I can't think of anything else offhand that I've installed.  I suspect
uninstalling it or more likely doing a fresh install will solve the problem but
I haven't tried it yet... at least not in a few weeks.

Comment 15

17 years ago
I just performed a fresh install of the latest nightly, and the menu worked
fine. I installed the preferences toolbar, and the menu no longer worked. I
reinstalled mozilla, and the bar works again. So whether installing an XPI is
the only cause, I don't know, but it *is* one cause.
I'm getting this as well. Feel free to drop by to see.


Comment 17

17 years ago
I did some testing and discovered the problem is that the personal toolbar
bookmarks button references its template using the template attribute, rather
than a template child.  The template it uses, bookmarksMenuTemplate, comes from
navigatorOverlay.xul.  Apparently, the xul content builder is looking for the
template before it is inserted by the overlay.  This doesn't always happen to
me, so it must be a timing issue.

I was able to solve the problem by moving the template directly into
navigator.xul.  I'm going to re-assign this bug to Ben because this is really
his domain, and I'll let him decide where to put the template.
Assignee: hewitt → ben
OK. Here's a patch that fixes the symptom and should get the menu running 
again. I couldn't reproduce this bug on any of my builds so I can't verify that 
it actually works, please let me know if it doesn't, I can think of several 

This doesn't fix the problem properly. A proposal for a better solution is 
described in a comment in this patch. If I can get approval to investigate that 
route, it should be followed and this code removed/ignored. 
The whitespace issue in the attached patch has been corrected. 

Comment 22

17 years ago
Comment on attachment 51852 [details] [diff] [review]

Attachment #51852 - Flags: review+

Comment 23

17 years ago
Comment on attachment 51852 [details] [diff] [review]

It's yucky, but it should work. sr=hewitt
Attachment #51852 - Flags: superreview+

Comment 24

17 years ago
Comment on attachment 51852 [details] [diff] [review]

a=asa (on behalf of drivers) for checkin to 0.9.5. hurry.
Attachment #51852 - Flags: approval+

Comment 25

17 years ago
Interesting.... when running Mozilla by itself (Build 2001100503) I see this
bug, no menu - BUT - when running Multizilla (v1.0.99E) on this build, the
bookmarks menu appears correctly.
Fixed checked in, branch & trunk,
Last Resolved: 17 years ago17 years ago
Resolution: --- → FIXED

Comment 27

17 years ago
Er...does anyone know what suddenly caused this, though?   The bookmarks button 
has been referencing a remote template by id forever.  If it was caused by 
hewitt's toolbarbutton changes, and I think it was, it'd be worth looking into 
Hewitt had toolbarbutton changes?

What did they involve?
HJ says this still isn't working right. 
Resolution: FIXED → ---

Comment 30

17 years ago
Correct, the first time it doesn't open/activate the bookmark popup.

WinNT4 with the build 10-08
I found that I think its the preferences bar that does it  (remember that it is
0.9.3 code and works on those builds but not properly on new ones), and if you
then install Multizilla you get it working while using Multizilla but not
Mozilla itself.. 

It you totally delete mozilla and your profile, and start a new install from
scratch this bug doesn't return.

/blame Pref Toolbar causes the problems not Mozilla ;)


Comment 32

17 years ago
What is 0.9.3 code, MultiZilla? Preference toolbar? Which preference toolbar are
you talking about? 

And FYI, MultiZilla is currently based on mozilla 0.9.4+ (builds after sept/30)
so no 0.9.3 there's no 0.9.3 code here and we are using 0.9.5 sources at this

0.9.3. man this guy is nuts.

Comment 33

17 years ago
This is definately either the mozgest toolbar or the UA toolbar for me. 
Summary: Personal Toolbar Bookmarks menu no longer works → PT Bookmarks menu stops working after installing an XPI add on


17 years ago
No longer blocks: 101793

Comment 34

17 years ago
0.9.5 is out the door. bumping TM up by one.
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Bumping down to P3, since the problem appears to be related to an unsupported

I'll remove the hacky code asap. 
Priority: P1 → P3

Comment 36

17 years ago
Ben said:
< ... related to an unsupported add-on. 

You are talking about mozgest or Multizilla? I mean, which on e is causing
problem and why. I'd be glad to correct this if it is in our part. Any
suggestion are appreciated.
--mondo (working on mozgest)
My comments were based on other comments further up where people claimed add-on
components were what "did it".. see cuz84d's comments

Comment 38

17 years ago
Ben: I'm not sure... Maybe I don't see things from the right angle, but IMHO
those ad-ons did not cause this bug. They just made it visible. (if I understood
corectly what was the problem here)
Keywords: helpwanted
Target Milestone: mozilla0.9.6 → mozilla0.9.9
Target Milestone: mozilla0.9.9 → mozilla1.1

Comment 39

16 years ago
the javascript error described in this bug is covered in bug 104360

Comment 40

16 years ago
Comment on attachment 51852 [details] [diff] [review]

obsoleting old checked in patches
Attachment #51852 - Attachment is obsolete: true

Comment 41

15 years ago
This patch should go in again for builds 20030325 and up. I had to e-mail HJ
about this bug and he told me to insert this removed code for MultiZilla and now
the bookmarks button works again.

-         <menupopup
+         <menupopup onpopupshowing="checkBookmarksMenuTemplateBuilder();

+// An interim workaround for 101131 - Bookmarks Toolbar button nonfunctional.
+// This simply checks to see if the bookmark menu is empty (aside from static
+// items) when it is opened and if it is, prompts a rebuild. 
+// The best fix for this is more time consuming, and relies on document
+// <template>s without content (referencing a remote <template/> by id) 
+// be noted as 'waiting' for a template to load from somewhere. When the 
+// ::Merge function in nsXULDocument is called and a template node inserted, 
+// the id of the template to be inserted is looked up in the map of waiting
+// references, and then the template builder hooked up. 
+function checkBookmarksMenuTemplateBuilder()
+  var lastStaticSeparator = document.getElementById("lastStaticSeparator");
+  if (!lastStaticSeparator.nextSibling) {
+    var button = document.getElementById("bookmarks-button");
+    button.builder.rebuild();
+  }


Comment 42

15 years ago
-> me
Assignee: ben → varga


15 years ago
Depends on: 199178

Comment 43

15 years ago
fixed along with bug 199178
Last Resolved: 17 years ago15 years ago
Resolution: --- → FIXED


10 years ago
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.