Right now existing windows don't get the menu until restart or a new window is created.

Bug 595096 was marked WONTFIX, and I think this should be too.

gavin: Not sure which bug you were referring to (you linked the dupe bug), and I have no strong opinions either way, but beltzner said "We should file a bug on that; it should appear instantly."

It's the right bug - it was marked WONTFIX before being marked DUPLICATE. Are you going to write a patch?

(In reply to comment #4) > It's the right bug - it was marked WONTFIX before being marked DUPLICATE. Oh, not sure why dao duped it this way then. > Are you going to write a patch? Like I said I have no strong opinions either way. It was requested by beltzner that a bug be filed. I suppose I could write a patch, but it's not my call if this is a blocker or will get approval for review cycles.

I'll review and approve it.

I started writing a test, but then got scared by non en-US issues of trying to trigger cmd-shift-j. Should I just check if the menuitem/command are correctly disabled/hidden?

(In reply to comment #7) > Should I just check if the menuitem/command are correctly disabled/hidden? Yes!

updated this to apply on trunk, and renamed "unloaders" -> "browserShutdownCallbacks".

Comment on attachment 478367 [details] [diff] [review] v1.1 >+ // Add an unloader to remove this pref observer >+ unloaders.push(function() gPrefService.removeObserver(pref, checkPref)); Don't land quite yet. This unloaders reference needs to be updated. I suppose my test can land separately.

Sigh, forgot to qref...

Comment on attachment 478377 [details] [diff] [review] for checkin >diff --git a/browser/base/content/test/browser_toggle_enabled.js b/browser/base/content/test/browser_toggle_enabled.js >+function _() dump(Array.join(arguments, " ") + "\n"); You shouldn't use dump() in a test - use info() if you really want output in the log, but I think these should just be comments.

Seems to be bitrotten by bug 601201 and probably will be by bug 602006.

Comment on attachment 478367 [details] [diff] [review] v1.1 (bitrotten)