221232 - [permmgr] js error in chrome://navigator/content/navigator.js

Status: VERIFIED FIXED

(Core :: Networking: Cookies, defect)

Description

[[e-mail]]

saw this while hacking around on cookiemgr today:

Error: popupManager.getEnumerator is not a function
Source File: chrome://navigator/content/navigator.js
Line: 172

looks like there's no getEnumerator method on nsIPopupWindowManager... mvl, is
this you?

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

I guess this is for firebird.
popupManager needs to be permissionManager. I'm not sure why the enumarator is
used. It seems to me that testPermission would work too, but i need to double
check that.

Comment 2

[[e-mail]]

firebird? no, i saw this on seamonkey.

please fix :)

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: patch v1

The popupwindowmanager enumerator is gone, the permission manager should be
used instead.

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 133572 [details] [diff] [review]
patch v1

on irc: <dwitte_z> r=me

bryner, can you sr?

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

checked in

Comment 6

[benc]

V:
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=navigator.js&branch=&root=/cvsroot&subdir=mozilla/xpfe/browser/resources/content&command=DIFF_FRAMESET&rev1=1.519&rev2=1.520

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

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