Don't mark element for style recalc when modifying its attribute style.
authorkling@webkit.org <kling@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2012 15:54:24 +0000 (15:54 +0000)
committerkling@webkit.org <kling@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Feb 2012 15:54:24 +0000 (15:54 +0000)
<http://webkit.org/b/78497>

Reviewed by Antti Koivisto.

StylePropertySet::setNeedsStyleRecalc() shouldn't do anything for attribute styles.
Their invalidation is handled exclusively by StyledElement::attributeChanged().
Elements with presentation attributes were being marked for style recalc twice,
once when the attribute changed, and again during attribute style update, below the
call to collectStyleForAttribute().

* css/StylePropertySet.cpp:
(WebCore::StylePropertySet::setNeedsStyleRecalc):

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

Source/WebCore/ChangeLog
Source/WebCore/css/StylePropertySet.cpp

index 1d7de59..d655060 100644 (file)
@@ -1,3 +1,19 @@
+2012-02-13  Andreas Kling  <awesomekling@apple.com>
+
+        Don't mark element for style recalc when modifying its attribute style.
+        <http://webkit.org/b/78497>
+
+        Reviewed by Antti Koivisto.
+
+        StylePropertySet::setNeedsStyleRecalc() shouldn't do anything for attribute styles.
+        Their invalidation is handled exclusively by StyledElement::attributeChanged().
+        Elements with presentation attributes were being marked for style recalc twice,
+        once when the attribute changed, and again during attribute style update, below the
+        call to collectStyleForAttribute().
+
+        * css/StylePropertySet.cpp:
+        (WebCore::StylePropertySet::setNeedsStyleRecalc):
+
 2012-02-13  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
 
         Move attribute storage from NamedNodeMap to ElementAttributeData
index 85f595a..6998e1a 100644 (file)
@@ -717,7 +717,8 @@ void StylePropertySet::setNeedsStyleRecalc()
             return;
 
         if (!m_isInlineStyleDeclaration) {
-            element->setNeedsStyleRecalc();
+            // If this is not an inline style, it's an attribute style, and we shouldn't mark the element for style recalc
+            // as that is handled by StyledElement::attributeChanged().
             return;
         }