[Invalidation Tracking] Trace StyleInvalidator setNeedsStyleRecalc

Source/core/inspector/InspectorTraceEvents.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/inspector/InspectorTraceEvents.h"
 
 #include "bindings/core/v8/ScriptCallStackFactory.h"
 #include "bindings/core/v8/ScriptGCEvent.h"
 #include "bindings/core/v8/ScriptSourceCode.h"
+#include "core/css/invalidation/DescendantInvalidationSet.h"
 #include "core/events/Event.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/inspector/IdentifiersFactory.h"
 #include "core/inspector/InspectorNodeIds.h"
 #include "core/inspector/ScriptCallStack.h"
 #include "core/page/Page.h"
 #include "core/rendering/RenderImage.h"
 #include "core/rendering/RenderLayer.h"
 #include "core/rendering/RenderObject.h"
 #include "core/workers/WorkerThread.h"
 #include "core/xml/XMLHttpRequest.h"
 #include "platform/JSONValues.h"
 #include "platform/TracedValue.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
 #include "platform/weborigin/KURL.h"
 #include "wtf/Vector.h"
+#include "wtf/text/StringBuilder.h"
 #include <inttypes.h>
 
 namespace blink {
 
 static const unsigned maxInvalidationTrackingCallstackSize = 5;
 
 namespace {
 
 class JSCallStack : public TraceEvent::ConvertableToTraceFormat  {
 public:
     explicit JSCallStack(PassRefPtrWillBeRawPtr<ScriptCallStack> callstack)
     {
         m_serialized = callstack ? callstack->buildInspectorArray()->toJSONString() : "[]";
         ASSERT(m_serialized.isSafeToSendToAnotherThread());
     }
     virtual String asTraceFormat() const
     {
         return m_serialized;
     }
 
 private:
     String m_serialized;
 };
 
 String toHexString(const void* p)
 {
     return String::format("0x%" PRIx64, static_cast<uint64>(reinterpret_cast<intptr_t>(p)));
 }
 
+String invalidationListToString(const WillBeHeapVector<RefPtrWillBeMember<DescendantInvalidationSet> >& invalidationList)
+{
+    StringBuilder builder;
+
+    builder.append("[");
+    bool isFirst = true;
+    for (const auto& invalidationSet : invalidationList) {
+        if (!isFirst)
+            builder.append(", ");
+        isFirst = false;
+
+        builder.append(invalidationSet->toString());
+    }
+    builder.append("]");
+    return builder.toString();
+}
+
 }
 
+const char InspectorStyleInvalidatorInvalidateEvent::ElementHasPendingInvalidationMap[] = "Element has pending invalidation map";
+const char InspectorStyleInvalidatorInvalidateEvent::InvalidateCustomPseudo[] = "Invalidate custom pseudo element.";
+const char InspectorStyleInvalidatorInvalidateEvent::InvalidationSetMatchedAttribute[] = "Invalidation set matched attribute.";
+const char InspectorStyleInvalidatorInvalidateEvent::InvalidationSetMatchedClass[] = "Invalidation set matched class.";
+const char InspectorStyleInvalidatorInvalidateEvent::InvalidationSetMatchedId[] = "Invalidation set matched id.";
+const char InspectorStyleInvalidatorInvalidateEvent::InvalidationSetMatchedTagName[] = "Invalidation set matched tagName.";
+const char InspectorStyleInvalidatorInvalidateEvent::PreventStyleSharingForParent[] = "Prevent style sharing for parent.";
+
+PassRefPtr<TraceEvent::ConvertableToTraceFormat> InspectorStyleInvalidatorInvalidateEvent::data(Element& element, const char* reason)
+{
+    return withExtraData(element, reason, nullAtom);
+}
+
+PassRefPtr<TraceEvent::ConvertableToTraceFormat> InspectorStyleInvalidatorInvalidateEvent::withExtraData(Element& element, const char* reason, const String& extraData)

[pdr., 2014/09/30 02:08:49]

Can you use a default arg here to avoid needing two functions?

... char* reason, const String& extraData = nullAtom) {

[kouhei (in TOK), 2014/09/30 02:22:00]

Acknowledged.

[kouhei (in TOK), 2014/09/30 03:44:05]

Done.

+{
+    RefPtr<TracedValue> value = TracedValue::create();

[pdr., 2014/09/30 03:21:57]

We need the frame info here too. Can you add:
value->setString("frame", toHexString(element->document().frame()));

[kouhei (in TOK), 2014/09/30 03:44:05]

Done.

+    value->setInteger("nodeId", InspectorNodeIds::idForNode(&element));

[pdr., 2014/09/30 02:08:50]

Can you use setNodeInfo here (from https://codereview.chromium.org/600733005)

[kouhei (in TOK), 2014/09/30 02:22:00]

Acknowledged.

[kouhei (in TOK), 2014/09/30 03:44:05]

Done.

+    value->setString("reason", reason);
+    value->setString("extraData", extraData);

[pdr., 2014/09/30 02:08:49]

Can we be more descriptive than just "extraData" (here, and elsewhere in this
patch)?

[kouhei (in TOK), 2014/09/30 02:22:00]

Agreed, but want your help on naming.

"extraData" is currently used to contain tagName/class/id/pseudo name that
caused invalidation.
- context?
- sourceName?

[pdr., 2014/09/30 03:21:57]

Talked offline a bit. We don't actually have the entire selector, only the last
matched part. Something like selectorInfo or selectorPart will probably be best.

[kouhei (in TOK), 2014/09/30 03:44:05]

Done.

+    return value;
+}
+
+PassRefPtr<TraceEvent::ConvertableToTraceFormat> InspectorStyleInvalidatorInvalidateEvent::invalidationList(Element& element, const WillBeHeapVector<RefPtrWillBeMember<DescendantInvalidationSet> >& invalidationList)
+{
+    return withExtraData(element, InspectorStyleInvalidatorInvalidateEvent::ElementHasPendingInvalidationMap, invalidationListToString(invalidationList));
+}
+
 PassRefPtr<TraceEvent::ConvertableToTraceFormat> InspectorLayoutEvent::beginData(FrameView* frameView)
 {
     bool isPartial;
     unsigned needsLayoutObjects;
     unsigned totalObjects;
     LocalFrame& frame = frameView->frame();
     frame.countObjectsNeedingLayout(needsLayoutObjects, totalObjects, isPartial);
 
     RefPtr<TracedValue> value = TracedValue::create();
     value->setInteger("dirtyObjects", needsLayoutObjects);
@@ skipping 331 matching lines @@
 
 PassRefPtr<TraceEvent::ConvertableToTraceFormat> InspectorTracingSessionIdForWorkerEvent::data(const String& sessionId, WorkerThread* workerThread)
 {
     RefPtr<TracedValue> value = TracedValue::create();
     value->setString("sessionId", sessionId);
     value->setDouble("workerThreadId", workerThread->platformThreadId());
     return value;
 }
 
 }