Summary: | Regression: querySelector matches tag names case sensitively | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Smith <catfish.man> | ||||
Component: | DOM | Assignee: | David Smith <catfish.man> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P1 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.5 | ||||||
URL: | http://dscoder.com/qsaidtest.html | ||||||
Attachments: |
|
Description
David Smith
2008-09-25 00:23:23 PDT
Created attachment 23784 [details]
Fix
Comment on attachment 23784 [details]
Fix
The change looks fine. I'd like to see an xhtml test as well (or maybe that's already covered?) That could also be done in a follow-up patch.
Comment on attachment 23784 [details]
Fix
+ std::auto_ptr<CSSSelector> parseSelector(const String&, Document* doc = 0);
The parameter should not be named here.
+std::auto_ptr<CSSSelector> CSSParser::parseSelector(const String& string, Document* doc)
+{
+ RefPtr<CSSStyleSheet> dummyStyleSheet = CSSStyleSheet::create(doc);
I strongly prefer "document" over "doc", but that's just my personal taste.
|