Closed Bug 520563 Opened 13 years ago Closed 13 years ago

Selector namespace support should not special-case the xml and xmlns prefixes

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.3a1

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file)

As the summary says.  Tests + patch coming up.
Presumably this applies to all CSS namespace support:  in selectors and attr().
OS: Mac OS X → All
Hardware: x86 → All
Attached patch Possible fixSplinter Review
This works, bug given that CSS is the only consumer, should we just rename, move, and simplify the class (e.g. to never insert xml and xmlns)?
Attachment #404637 - Flags: review?(jonas)
Attachment #404637 - Flags: review?(dbaron)
> Presumably this applies to all CSS namespace support:  in selectors and attr().

Yes, it does.
Comment on attachment 404637 [details] [diff] [review]
Possible fix

r=dbaron
Attachment #404637 - Flags: review?(dbaron) → review+
Pushed http://hg.mozilla.org/mozilla-central/rev/48c4226236eb
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
You need to log in before you can comment on or make changes to this bug.