Track reasons for |Node::SetNeedsStyleRecalc|

Source/core/dom/TreeScope.cpp

 /*
  * Copyright (C) 2011 Google Inc. All Rights Reserved.
  * Copyright (C) 2012 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 524 matching lines @@
     return result;
 }
 
 void TreeScope::setNeedsStyleRecalcForViewportUnits()
 {
     for (Element* element = ElementTraversal::firstWithin(rootNode()); element; element = ElementTraversal::nextIncludingPseudo(*element)) {
         for (ShadowRoot* root = element->youngestShadowRoot(); root; root = root->olderShadowRoot())
             root->setNeedsStyleRecalcForViewportUnits();
         RenderStyle* style = element->renderStyle();
         if (style && style->hasViewportUnits())
-            element->setNeedsStyleRecalc(LocalStyleChange);
+            element->setNeedsStyleRecalc(StyleChangeReasonForTracing::create(StyleChangeReason::ViewportUnits), LocalStyleChange);

[esprehn, 2014/09/26 03:53:01]

Can we make this the second argument instead? Now when I'm reading the
setNeedsStyleRecalc callsites the important behavioral code is all the way at
the far end of the screen.

[kouhei (in TOK), 2014/09/29 02:05:32]

Done.

     }
 }
 
 void TreeScope::trace(Visitor* visitor)
 {
     visitor->trace(m_rootNode);
     visitor->trace(m_document);
     visitor->trace(m_parentTreeScope);
     visitor->trace(m_idTargetObserverRegistry);
     visitor->trace(m_selection);
     visitor->trace(m_elementsById);
     visitor->trace(m_imageMapsByName);
     visitor->trace(m_labelsByForAttribute);
     visitor->trace(m_scopedStyleResolver);
 }
 
 } // namespace blink