Track reasons for |Node::SetNeedsStyleRecalc|

Source/core/dom/Document.cpp

Index: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index ac3d14065363a169bfbaab9e2a1207145b4f692b..eafbbf15a0934a8e2458b0f7bc60d0dd5b2eb904 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -1205,7 +1205,7 @@ void Document::setContentLanguage(const AtomicString& language)
     m_contentLanguage = language;
 
     // Document's style depends on the content language.
-    setNeedsStyleRecalc(SubtreeStyleChange);
+    setNeedsStyleRecalc(StyleRecalcDueToLanguage, SubtreeStyleChange);
 }
 
 void Document::setXMLVersion(const String& version, ExceptionState& exceptionState)
@@ -1736,7 +1736,7 @@ void Document::inheritHtmlAndBodyElementStyles(StyleRecalcChange change)
     // rare and just invalidate the cache for now.
     if (styleEngine()->usesRemUnits() && (documentElement()->needsAttach() || documentElement()->computedStyle()->fontSize() != documentElementStyle->fontSize())) {
         ensureStyleResolver().invalidateMatchedPropertiesCache();
-        documentElement()->setNeedsStyleRecalc(SubtreeStyleChange);
+        documentElement()->setNeedsStyleRecalc(StyleRecalcDueToFontSizeChange, SubtreeStyleChange);
     }
 
     EOverflow overflowX = OAUTO;
@@ -1774,13 +1774,13 @@ void Document::inheritHtmlAndBodyElementStyles(StyleRecalcChange change)
     if (body) {
         if (RenderStyle* style = body->renderStyle()) {
             if (style->direction() != rootDirection || style->writingMode() != rootWritingMode)
-                body->setNeedsStyleRecalc(SubtreeStyleChange);
+                body->setNeedsStyleRecalc(StyleRecalcDueToWritingModeChange, SubtreeStyleChange);
         }
     }
 
     if (RenderStyle* style = documentElement()->renderStyle()) {
         if (style->direction() != rootDirection || style->writingMode() != rootWritingMode)
-            documentElement()->setNeedsStyleRecalc(SubtreeStyleChange);
+            documentElement()->setNeedsStyleRecalc(StyleRecalcDueToWritingModeChange, SubtreeStyleChange);
     }
 }
 
@@ -1963,7 +1963,7 @@ void Document::updateLayout()
 
 void Document::setNeedsFocusedElementCheck()
 {
-    setNeedsStyleRecalc(LocalStyleChange);
+    setNeedsStyleRecalc(StyleRecalcDueToFocusPseudoClass, LocalStyleChange);
 }
 
 void Document::clearFocusedElementSoon()
@@ -2089,7 +2089,7 @@ bool Document::dirtyElementsForLayerUpdate()
         return false;
 
     for (WillBeHeapHashSet<RawPtrWillBeMember<Element> >::iterator it = m_layerUpdateSVGFilterElements.begin(), end = m_layerUpdateSVGFilterElements.end(); it != end; ++it)
-        (*it)->setNeedsStyleRecalc(LocalStyleChange);
+        (*it)->setNeedsStyleRecalc(StyleRecalcDueToSVGFilterLayerUpdateHack, LocalStyleChange);

[pdr., 2014/09/06 22:36:27]

We should probably just call this one StyleRecalcDueToSVGFilterLayerUpdate.

[kouhei (in TOK), 2014/09/08 23:29:12]

Done.

     m_layerUpdateSVGFilterElements.clear();
     return true;
 }
@@ -4315,7 +4315,7 @@ void Document::setEncodingData(const DocumentEncodingData& newData)
         // FIXME: How is possible to not have a renderer here?
         if (renderView())
             renderView()->style()->setRTLOrdering(m_visuallyOrdered ? VisualOrder : LogicalOrder);
-        setNeedsStyleRecalc(SubtreeStyleChange);
+        setNeedsStyleRecalc(StyleRecalcDueToVisuallyOrdered, SubtreeStyleChange);
     }
 }
 
@@ -4477,7 +4477,7 @@ void Document::setDesignMode(InheritedBool value)
             continue;
         if (!toLocalFrame(frame)->document())
             break;
-        toLocalFrame(frame)->document()->setNeedsStyleRecalc(SubtreeStyleChange);
+        toLocalFrame(frame)->document()->setNeedsStyleRecalc(StyleRecalcDueToDesignMode, SubtreeStyleChange);
     }
 }