Fix for 3787686, CSS parser should do case-sensitive matching of ids and attributes...
authorhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Sep 2004 17:28:02 +0000 (17:28 +0000)
committerhyatt <hyatt@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Sep 2004 17:28:02 +0000 (17:28 +0000)
(and not whenever you have an HTML document).

        Reviewed by john

        * khtml/css/parser.cpp:
        * khtml/css/parser.y:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7443 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/css/parser.cpp
WebCore/khtml/css/parser.y

index 0dfc03d778915556a18f36b658fd49887b46d19c..42d73fda8d7e989f0b1d37a3c58b95465fc8fc64 100644 (file)
@@ -1,3 +1,13 @@
+2004-09-03  David Hyatt  <hyatt@apple.com>
+
+       Fix for 3787686, CSS parser should do case-sensitive matching of ids and attributes only in strict mode
+       (and not whenever you have an HTML document).
+       
+        Reviewed by john
+
+        * khtml/css/parser.cpp:
+        * khtml/css/parser.y:
+
 === Safari-162 ===
 
 2004-09-02  David Hyatt  <hyatt@apple.com>
index 4f5d6501b6de32bd174df579bdcaeaf2a37277b0..061089b32f89ee9f7912e9baaa6e3bb5ee0e8d31 100644 (file)
@@ -1565,7 +1565,7 @@ case 75:
        DOM::DocumentImpl *doc = p->document();
        QString tag = qString(yyvsp[0].string);
        if ( doc ) {
-           if (doc->isHTMLDocument())
+           if (!p->strict)
                tag = tag.lower();
            const DOMString dtag(tag);
             yyval.element = makeId(p->defaultNamespace, doc->tagId(0, dtag.implementation(), false));
@@ -1641,7 +1641,7 @@ case 85:
 
        QString attr = qString(yyvsp[-1].string);
        if ( doc ) {
-           if (doc->isHTMLDocument())
+           if (!p->strict)
                attr = attr.lower();
            const DOMString dattr(attr);
             yyval.attribute = doc->attrId(0, dattr.implementation(), false);
index f1428753d3ee73cedfdc2bdb3ca86a4cddadb659..b7d609a4b4536638e0b98c3f43a1ead9adbd5bc6 100644 (file)
@@ -631,7 +631,7 @@ element_name:
        DOM::DocumentImpl *doc = p->document();
        QString tag = qString($1);
        if ( doc ) {
-           if (doc->isHTMLDocument())
+           if (!p->strict)
                tag = tag.lower();
            const DOMString dtag(tag);
             $$ = makeId(p->defaultNamespace, doc->tagId(0, dtag.implementation(), false));
@@ -701,7 +701,7 @@ attrib_id:
 
        QString attr = qString($1);
        if ( doc ) {
-           if (doc->isHTMLDocument())
+           if (!p->strict)
                attr = attr.lower();
            const DOMString dattr(attr);
             $$ = doc->attrId(0, dattr.implementation(), false);