Closed
Bug 734714
Opened 12 years ago
Closed 12 years ago
Some prompts broken: "aDomWin is null" in prompt service
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(firefox15 verified, blocking-fennec1.0 beta+)
VERIFIED
FIXED
Firefox 13
People
(Reporter: zac..luzader.+bugzilla, Assigned: mfinkle)
References
Details
Attachments
(2 files)
7.12 KB,
text/plain
|
Details | |
1.11 KB,
patch
|
wesj
:
review+
|
Details | Diff | Splinter Review |
The "add search engine" prompt (the one triggered by window.external.AddSearchProvider) and a dialog created by an addon (mbrubeck's "phony" -- user agent switcher) don't work. No prompt is displayed. alogcat shows the exception: aDomWin is null on line 777 of /components/PromptService.js Log snippets attached. Log taken on an Ice Cream Sandwich tablet, but I can also duplicate it on an Android 2.3 Motorola Atrix, both running 2010-03-10 Nightly.
Assignee | ||
Comment 1•12 years ago
|
||
Regression from bug 711624. We need to check for a null domWin like desktop does here: http://mxr.mozilla.org/mozilla-central/source/toolkit/components/prompts/src/nsPrompter.js#493 Doing a null check for domWin here should do it: http://mxr.mozilla.org/mozilla-central/source/mobile/android/components/PromptService.js#159
Assignee: nobody → margaret.leibovic
blocking-fennec1.0: --- → beta+
Assignee | ||
Comment 2•12 years ago
|
||
simple one-liner. testing with phony as soon as the build is finished.
Assignee: margaret.leibovic → mark.finkle
Attachment #604745 -
Flags: review?(margaret.leibovic)
Assignee | ||
Updated•12 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Assignee | ||
Comment 3•12 years ago
|
||
patch worked fine with phony
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 604745 [details] [diff] [review] patch Margaret is on PTO
Attachment #604745 -
Flags: review?(margaret.leibovic) → review?(wjohnston)
Comment 5•12 years ago
|
||
Comment on attachment 604745 [details] [diff] [review] patch Review of attachment 604745 [details] [diff] [review]: ----------------------------------------------------------------- This is what we did in XUL Fennec, and I think its fine. Desktop actually tries much harder to get a window. See: http://mxr.mozilla.org/mozilla-central/source/toolkit/components/prompts/src/nsPrompter.js#493 which winds up calling WindowWatcher which does... a whole lot to ensure they find something. I... don't think its worth that much effort right now.
Attachment #604745 -
Flags: review?(wjohnston) → review+
Updated•12 years ago
|
Priority: -- → P2
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/261d255eb06d
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/261d255eb06d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 13
Comment 9•12 years ago
|
||
Verified fixed on: Firefox 15.0a1 (2012-04-30) Device: Samsung Captivate OS: Android 2.2
Status: RESOLVED → VERIFIED
status-firefox15:
--- → verified
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•