[Invalidation Tracking] Trace StyleInvalidator setNeedsStyleRecalc

Source/core/inspector/InspectorTraceEvents.h

Index: Source/core/inspector/InspectorTraceEvents.h
diff --git a/Source/core/inspector/InspectorTraceEvents.h b/Source/core/inspector/InspectorTraceEvents.h
index 24ffa7165b1b645695ea0218bd59f291906ed4e1..10fa622aae0d948642cb8e52c5da41f1f6738eca 100644
--- a/Source/core/inspector/InspectorTraceEvents.h
+++ b/Source/core/inspector/InspectorTraceEvents.h
@@ -7,11 +7,14 @@
 
 #include "platform/EventTracer.h"
 #include "platform/TraceEvent.h"
+#include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 
 namespace blink {
 
+class DescendantInvalidationSet;
 class Document;
+class Element;
 class Event;
 class ExecutionContext;
 class FrameView;
@@ -36,6 +39,32 @@ public:
     static PassRefPtr<TraceEvent::ConvertableToTraceFormat> endData(RenderObject* rootForThisLayout);
 };
 
+class InspectorStyleInvalidatorInvalidateEvent {

[caseq, 2014/09/30 12:24:00]

nit: here and below, I wonder if we need both Invalidator and Invalidate in one
identifier. InspectorStyleInvalidationEvent perhaps?

[kouhei (in TOK), 2014/10/01 04:16:14]

.
I think StyleInvalidatorInvalidate is correct here. This is recalcStyle
invalidation caused by StyleInvalidator, which precedes 
InspectorStyleRecalcInvalidationTrackingEvent.

+public:
+    static const char ElementHasPendingInvalidationMap[];
+    static const char InvalidateCustomPseudo[];
+    static const char InvalidationSetMatchedAttribute[];
+    static const char InvalidationSetMatchedClass[];
+    static const char InvalidationSetMatchedId[];
+    static const char InvalidationSetMatchedTagName[];
+    static const char PreventStyleSharingForParent[];
+
+    static PassRefPtr<TraceEvent::ConvertableToTraceFormat> data(Element&, const char* reason, const String& = nullAtom);
+    static PassRefPtr<TraceEvent::ConvertableToTraceFormat> invalidationList(Element&, const WillBeHeapVector<RefPtrWillBeMember<DescendantInvalidationSet> >&);
+};
+#define TRACE_STYLE_INVALIDATOR_INVALIDATION(ELEMENT, REASON) \

[caseq, 2014/09/30 12:24:00]

nit: plase add a blank line above the macro definition. Also, lowercase the
macro parameters, they normally follow function arguments style.

[kouhei (in TOK), 2014/10/01 04:16:14]

Done.

+    TRACE_EVENT_INSTANT1( \
+        TRACE_DISABLED_BY_DEFAULT("devtools.timeline.invalidationTracking"), \
+        "StyleInvalidatorInvalidationTracking", \
+        "data", \
+        InspectorStyleInvalidatorInvalidateEvent::data((ELEMENT), (InspectorStyleInvalidatorInvalidateEvent::REASON)))
+#define TRACE_STYLE_INVALIDATOR_INVALIDATION_SELECTORPART(ELEMENT, REASON, SELECTORPART) \

[caseq, 2014/09/30 12:24:00]

ditto

[kouhei (in TOK), 2014/10/01 04:16:15]

Done.

+    TRACE_EVENT_INSTANT1( \
+        TRACE_DISABLED_BY_DEFAULT("devtools.timeline.invalidationTracking"), \
+        "StyleInvalidatorInvalidationTracking", \
+        "data", \
+        InspectorStyleInvalidatorInvalidateEvent::data((ELEMENT), (InspectorStyleInvalidatorInvalidateEvent::REASON), (SELECTORPART)))
+
 class InspectorLayoutInvalidationTrackingEvent {
 public:
     static PassRefPtr<TraceEvent::ConvertableToTraceFormat> data(const RenderObject*);