Closed Bug 595507 Opened 10 years ago Closed 10 years ago

When an invalid form in an IFRAME is submitted, error messages are not displayed

Categories

(Firefox :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 4.0b7
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: jk1700, Assigned: mounir)

References

()

Details

(Keywords: html5)

Attachments

(1 file, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100911 Firefox/4.0b6pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b6pre) Gecko/20100911 Firefox/4.0b6pre

When the page contains an IFRAME with the form containing required fields the error message "This field is required" is not displayed

Reproducible: Always

Steps to Reproduce:
1. Open the page http://files.karczmarczyk.pl/x/
2. Don't fill in the text input, just click "Submit"

Actual Results:  
Nothing happens

Expected Results:  
Error message "This field is required" should be displayed

When you open the iframe page http://files.karczmarczyk.pl/x/iframe.html the error message is displayed
Blocks: 344614
Keywords: html5
Version: unspecified → Trunk
Nice catch ;)

I will investigate that. I think this should probably be a blocker considering this will give a very bad experience to our users.
Assignee: nobody → mounir.lamouri
Blocks: 561636
No longer blocks: 344614
Status: UNCONFIRMED → NEW
blocking2.0: --- → ?
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86_64 → All
Assignee: mounir.lamouri → nobody
Component: General → DOM: Core & HTML
Product: Firefox → Core
QA Contact: general → general
Oops - didn't mean to reset assignee.
Assignee: nobody → mounir.lamouri
I do not think that's a dom bug actually. The message is shown by browser.js.
Blocking based on comment 1.
blocking2.0: ? → betaN+
Status: NEW → ASSIGNED
Component: DOM: Core & HTML → General
Product: Core → Firefox
QA Contact: general → general
Attached patch Patch v1 (obsolete) — Splinter Review
Attachment #475987 - Flags: review?(dolske)
Attached patch Patch v1.1Splinter Review
Oups, qnew put some other changes inside the patch. Should be better now.
Attachment #475987 - Attachment is obsolete: true
Attachment #475988 - Flags: review?(dolske)
Attachment #475987 - Flags: review?(dolske)
Comment on attachment 475988 [details] [diff] [review]
Patch v1.1

r=me if you also change "gBrowser.selectedTab.linkedBrowser.contentDocument" to "gBrowser.contentDocument" :)
Attachment #475988 - Flags: review?(dolske) → review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/2a2ec6c26acf
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b7
You need to log in before you can comment on or make changes to this bug.