Closed Bug 235531 Opened 20 years ago Closed 20 years ago

cookie/image manager menu in Tools can't work

Categories

(Core :: Networking: Cookies, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: iamawalrus, Assigned: mconnor)

Details

(Keywords: regression)

Attachments

(1 file)

1. open a url in the browser
2. open the javascript console
3. click Tools->Image Manager-><Item that has not been checked>
result:
an error in the javascript consle as:
Error: permissionmanager has no properties
Source File: chrome://cookie/content/cookieNavigatorOverlay.xul
Line: 132
The message box of "permission changed" does not prompt out and the url has not
been added to the manager site lists.

mozilla1.7a does not have the problem. I found it in the lastest trunk which is
due to the onpopupshowing of the menupouup in cookieNavigatorOverlay.xul being
not triggered.
tested this before I left for work, same thing on yesterday's Win32 trunk build.

-> me, should have a fix up tonight once I figure out what happened
Assignee: darin → mconnor
Keywords: regression
OS: other → All
Hardware: Other → All
the checkin for bug 142961 broke this by adding a second onpopupshowing to
taskPopup.

Neil, I remember you complaining about this being a Bad Thing(tm) in a patch for
cookieNavigatorOverlay.xul, yet you let someone add another one?  Maybe we
should move the onpopupshowing="CheckForVisibility()" onto the submenu menupopup
elements to prevent this from happening?  The only concern is that this means
this will get called a fair number of times more, but how much will it affect
real-world perf?
Attached patch proposed fixSplinter Review
Comment on attachment 142417 [details] [diff] [review]
proposed fix

this will result in a call to initWalletMenuItems() each time the submenu is
shown, but I don't think this is anywhere close to affecting real-world
performance
Attachment #142417 - Flags: superreview?(alecf)
Attachment #142417 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 142417 [details] [diff] [review]
proposed fix

This is exactly what should have gone in, sorry for not picking up on this in
the previous patch :-[
Attachment #142417 - Flags: review?(neil.parkwaycc.co.uk) → review+
Attachment #142417 - Flags: superreview?(alecf) → superreview?(bzbarsky)
Comment on attachment 142417 [details] [diff] [review]
proposed fix

sr=bzbarsky
Attachment #142417 - Flags: superreview?(bzbarsky) → superreview+
checked in 02/29/2004 09:48
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.