Closed Bug 766481 Opened 11 years ago Closed 11 years ago
.alert/prompt/confirm in <iframe> within <iframe mozbrowser>
+++ This bug was initially created as a clone of Bug #742944 +++ Gaia issue: https://github.com/mozilla-b2g/gaia/issues/1802 For HTML structure like this: <iframe mozbrower> <iframe> <button onclick="alert('hi')">Alert</button> </iframe> </iframe> The alert() call can bypass the mozbrowser capability implemented in Bug #742944 STR: Launch Gaia "UI Tests" app and try the alert/prompt test.
Summary: Handle window.open in <iframe mozbrowser> → Handle window.open in <iframe> within <iframe mozbrowser>
Is the problem alert() or window.open? Or do both not work within <iframe>s?
Also, these dependencies / blocking bugs are probably not what you mean, but I'm not sure what you mean for this bug to cover, so I don't want to change them.
@jlebar Yeah I totally missed up with the dependency thing ... I've updated the title. Can you fix the dependency? Thanks.
Summary: Handle window.open in <iframe> within <iframe mozbrowser> → Handle window.alert/prompt/confirm in <iframe> within <iframe mozbrowser>
blocking-basecamp: + → ---
blocking-kilimanjaro: + → ---
This was a pretty dumb bug. Oh well.
Attachment #635171 - Flags: review?(mounir)
Assignee: nobody → justin.lebar+bug
Component: General → DOM: Mozilla Extensions
Product: Boot2Gecko → Core
QA Contact: general → general
Even better; now it's a one-line fix.
Attachment #636066 - Flags: review?(mounir) → review+
Attachment #635171 - Flags: review?(mounir) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
What is the change to nsGenericElement.cpp ? https://hg.mozilla.org/mozilla-central/rev/f3cd319dd52d
(In reply to Olli Pettay [:smaug] from comment #8) > What is the change to nsGenericElement.cpp ? > https://hg.mozilla.org/mozilla-central/rev/f3cd319dd52d Known bug in hg, although I can't find the bug at the moment. It seems to be harmless, although certainly quite scary.
Component: DOM: Mozilla Extensions → DOM
You need to log in before you can comment on or make changes to this bug.