Add about:blank to my above list of detectable problem URLs. http://www.chicagosuntimes.com/index/ebert.html has a popup window like that in the opening comment, except it's opened initially to about:blank, rather than a 0-length string.
Is the cnn.com popup mentioned in bug 176564, comment #26 one of those? Or is that another problem?
mvl and danm, think we can address this in the next few weeks?
Created attachment 160943 [details] [diff] [review] omit unopenable items from popup menu (Aviary) This patch addresses the issue by simply not offering to open windows for which a useful URL was not supplied in the window.open command, as outlined in comment 0. I think it's all we can do, practically speaking.
Comment on attachment 160943 [details] [diff] [review] omit unopenable items from popup menu (Aviary) a=asa for aviary checkin.
Attachment #160943 - Flags: approval-aviary? → approval-aviary+
Whiteboard: trunk prerequisite: bug 250716 → trunk req: 250716
So, what is there left to do here?
i'll leave it to the component owner to decide if they want this.
Assignee: mvl → guifeatures
How about not showing the separator in the end of the menu when the show option is not visible?
Filter "spam" on "guifeatures-nobody-20080610".
Assignee: guifeatures → nobody
QA Contact: guifeatures
You need to log in before you can comment on or make changes to this bug.