Closed Bug 125647 Opened 23 years ago Closed 23 years ago

input type=radio checked attribute problem

Categories

(Core :: DOM: HTML Parser, defect)

x86
All
defect
Not set
minor

Tracking

()

VERIFIED FIXED
mozilla1.0

People

(Reporter: mrcawfee, Assigned: harishd)

References

()

Details

Attachments

(2 files)

From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.8) Gecko/20020204 BuildID: 2002020406 If you have a radio button like this <input type="radio" name="button" value="1" CHECKED/> the browser will not mark the button as checked it will ony do so if it is <input type="radio" name="button" CHECKED value="1"/> Reproducible: Always Steps to Reproduce: 1.make a page with a form and try both of thosecombonations 2. 3.
Attached file testcase (deleted) —
Status: UNCONFIRMED → NEW
Component: Browser-General → HTML Form Controls
Ever confirmed: true
OS: Windows 2000 → All
Summary: form checked errors → input type=radio checked attribute problem
doh! really moving
Assignee: asa → rods
QA Contact: doronr → madhur
-->
Assignee: rods → jkeiser
I have definitely dealt with this issue in bug 108308 (it was frustrating).
Depends on: 108308
Keywords: nsbeta1
Target Milestone: --- → mozilla1.0
Marking nsbeta1+ since it will be solved by the patch for bug 108308
Keywords: nsbeta1nsbeta1+
Hmm, it appears the only thing that is wrong with this is a parser issue. CHECKED /> is being parsed but CHECKED/> is not.
Assignee: jkeiser → harishd
Component: HTML Form Controls → Parser
QA Contact: madhur → moied
No longer depends on: 108308
Attached patch patch v1.0 (deleted) — Splinter Review
The forward slash in an start tag, <TAG ATTR/>, should not be a part of the attribute. When trying to do that we swallowed, in addition to '/', the alpha character preceding the forward slash. The fix is to update the iterator position such that the legal attribute is not truncated.
Comment on attachment 73985 [details] [diff] [review] patch v1.0 sr=jst
Attachment #73985 - Flags: superreview+
Comment on attachment 73985 [details] [diff] [review] patch v1.0 a=scc
Attachment #73985 - Flags: approval+
FIXED
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
verified fix checked in cvs (rev 3.213)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: