Closed Bug 401612 Opened 17 years ago Closed 17 years ago

Multiple dialogs (about, add-ons mgr, etc.) will not open (nsIDOMJSWindow.openDialog exception)

Categories

(Core :: General, defect)

x86
Linux
defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla1.9beta1

People

(Reporter: reed, Assigned: peterv)

References

Details

(Keywords: regression)

Attachments

(1 file)

Error: uncaught exception: [Exception... "Component returned failure code: 0xb78c6dba [nsIDOMJSWindow.openDialog]" nsresult: "0xb78c6dba (<unknown>)" location: "JS frame :: chrome://browser/content/utilityOverlay.js :: openAboutDialog :: line 318" data: no] Regressed sometime today, I think.
Flags: blocking-firefox3?
From trying to open the add-ons manager: Error: uncaught exception: [Exception... "Component returned failure code: 0xb77dd98a [nsIDOMJSWindow.openDialog]" nsresult: "0xb77dd98a (<unknown>)" location: "JS frame :: chrome://browser/content/preferences/main.js :: anonymous :: line 430" data: no] Pretty sure peterv's patch caused this.
Flags: blocking-firefox3?
Product: Firefox → Core
QA Contact: general → general
Summary: About Dialog will not open (nsIDOMJSWindow.openDialog exception) → Multiple dialogs (about, add-ons mgr, etc.) will not open (nsIDOMJSWindow.openDialog exception)
Target Milestone: Firefox 3 M9 → mozilla1.9 M9
Flags: blocking1.9?
Appears to be Linux-only, as this affects my Ubuntu Fiesty trunk build, but neither Mac OS X 10.4 nor Windows Vista.
Blocks: 379718
Attached patch v1 (deleted) — Splinter Review
Trivial fix, always initialize the result argument :-(.
Assignee: nobody → peterv
Status: NEW → ASSIGNED
Attachment #286660 - Flags: superreview?
Attachment #286660 - Flags: review?
Attachment #286660 - Flags: approvalM9?
confirmed, patch fixes issue.
Comment on attachment 286660 [details] [diff] [review] v1 Um, the constructor is a bit ugly. But it was ugly even before your changes.
Attachment #286660 - Flags: superreview?(jst)
Attachment #286660 - Flags: superreview?
Attachment #286660 - Flags: review?
Attachment #286660 - Flags: review+
Flags: blocking1.9? → blocking1.9+
Comment on attachment 286660 [details] [diff] [review] v1 jst: can you sr? this? It's now a M9 blocker so doesn't need explicit aM9+
Attachment #286660 - Flags: approvalM9?
Attachment #286660 - Flags: superreview?(jst) → superreview+
Peter: If you can, please commit this before nightlies tonight. If you don't have time to or don't want to watch the tree, you can just add the checkin-needed keyword, and somebody will do it for you. Thanks! I really appreciate it. :)
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
verified fixed using Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a9pre) Gecko/2007110106 Minefield/3.0a9pre Open Dialogs (about, add-ons mgr) is now working fine -> Verified
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: