Closed Bug 789721 Opened 10 years ago Closed 10 years ago

Check dom.mozBrowserFramesEnabled before BrowserElementPromptService initialized


(Firefox OS Graveyard :: General, defect)

Not set


(Not tracked)



(Reporter: kk1fff, Assigned: kk1fff)




(1 file, 1 obsolete file)

No description provided.
Attached patch Patch (obsolete) — Splinter Review
To prevent from installing wrapped BrowserElementPromptService in application that don't use mozbrowser.

Pushed to try
Attachment #659516 - Flags: review?(justin.lebar+bug)
Mark, can you verify that this fixes the problem for you?  If the commit mentioned in bug 789572 is actually the problem, I'd be surprised if this fixed anything.
Comment on attachment 659516 [details] [diff] [review]

Please add code to ensure that we don't run _init() twice.

r=me with that change, but I'd like someone to verify that this actually fixes the problem.
Attachment #659516 - Flags: review?(justin.lebar+bug)
Attachment #659516 - Flags: review+
Attachment #659516 - Flags: feedback?(mark.finkle)
Attached patch PatchSplinter Review
Attachment #659516 - Attachment is obsolete: true
Attachment #659516 - Flags: feedback?(mark.finkle)
Attachment #659546 - Flags: review?(justin.lebar+bug)
Attachment #659546 - Flags: feedback?(mark.finkle)
Comment on attachment 659546 [details] [diff] [review]

Maybe this is a nit, but I'd move the |this._initialized = true| line earlier in _init() because if we throw  an exception in the middle of _init(), I think we should just give up, rather than try to re-initialize when the pref changes again.

r+ means you don't need to re-request review on the next patch, but I would prefer to hear from mfinkle if this solves their problem before checking it in.
Attachment #659546 - Flags: review?(justin.lebar+bug) → review+
I tested the Try build and it seems to work fine. I get the prompt when trying to install a webapp and when trying to install an add-on
Attachment #659546 - Flags: feedback?(mark.finkle) → feedback+
Since it's a trivial change and we want to get it in the next nightly, I've pushed Patrick's patch with my last nit addressed:
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.