Changeset 101205 in webkit

Timestamp:

Nov 27, 2011, 12:02:25 PM (15 years ago)

Author:

commit-queue@webkit.org

Message:

Add assertions that CSSMappedAttributeDeclarations aren't using strict parsing.
<​http://webkit.org/b/73167>

Patch by Andreas Kling <​kling@webkit.org> on 2011-11-27
Reviewed by Antti Koivisto.

• dom/StyledElement.cpp:

(WebCore::StyledElement::createMappedDecl):

• html/HTMLTableElement.cpp:

(WebCore::HTMLTableElement::additionalAttributeStyleDecls):
(WebCore::HTMLTableElement::addSharedCellBordersDecl):
(WebCore::HTMLTableElement::addSharedCellPaddingDecl):
(WebCore::HTMLTableElement::addSharedGroupDecls):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/StyledElement.cpp (modified) (1 diff)
• html/HTMLTableElement.cpp (modified) (4 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-11-27  Andreas Kling  <kling@webkit.org>
+
+        Add assertions that CSSMappedAttributeDeclarations aren't using strict parsing.
+        <http://webkit.org/b/73167>
+
+        Reviewed by Antti Koivisto.
+
+        * dom/StyledElement.cpp:
+        (WebCore::StyledElement::createMappedDecl):
+        * html/HTMLTableElement.cpp:
+        (WebCore::HTMLTableElement::additionalAttributeStyleDecls):
+        (WebCore::HTMLTableElement::addSharedCellBordersDecl):
+        (WebCore::HTMLTableElement::addSharedCellPaddingDecl):
+        (WebCore::HTMLTableElement::addSharedGroupDecls):
+
 2011-11-27  Andreas Kling  <kling@webkit.org>
 

trunk/Source/WebCore/dom/StyledElement.cpp

@@ -408 +408 @@
     decl->setParentStyleSheet(document()->elementSheet());
     decl->setElement(this);
+    ASSERT(!decl->useStrictParsing());
 }
 

trunk/Source/WebCore/html/HTMLTableElement.cpp

@@ -458 +458 @@
         decl->setParentStyleSheet(document()->elementSheet());
         decl->setElement(this);
+        ASSERT(!decl->useStrictParsing());
         
         int v = m_borderColorAttr ? CSSValueSolid : CSSValueOutset;
@@ -515 +516 @@
         decl->setParentStyleSheet(document()->elementSheet());
         decl->setElement(this);
+        ASSERT(!decl->useStrictParsing());
         
         switch (borders) {
@@ -573 +575 @@
             m_paddingDecl->setParentStyleSheet(document()->elementSheet());
             m_paddingDecl->setElement(this);
+            ASSERT(!m_paddingDecl->useStrictParsing());
             
             m_paddingDecl->setProperty(CSSPropertyPaddingTop, paddingValue, false);
@@ -599 +602 @@
         decl->setParentStyleSheet(document()->elementSheet());
         decl->setElement(this);
+        ASSERT(!decl->useStrictParsing());
         
         if (rows) {