CSS: Additional asserts for style invalidation

third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h

Index: third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h
diff --git a/third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h b/third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h
index bf918e425a3048bbba0db60a13b0ea57f6dcb6cb..19b6b1d547a23894f92250841083b186b1a02479 100644
--- a/third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h
+++ b/third_party/WebKit/Source/core/css/invalidation/InvalidationSet.h
@@ -80,7 +80,7 @@ enum InvalidationType {
 // We avoid virtual functions to minimize space consumption.
 class CORE_EXPORT InvalidationSet {
     WTF_MAKE_NONCOPYABLE(InvalidationSet);
-    USING_FAST_MALLOC_WITH_TYPE_NAME(blink::InvalidationSet);
+    USING_FAST_MALLOC(blink::InvalidationSet);

[esprehn, 2016/09/01 05:30:22]

Why change this in this patch? The other macro seems about the same, I'm also
not sure why we annotate InvalidationSet and CSSSelector with
USING_FAST_MALLOC_WITH_TYPE_NAME but nothing else.

[Eric Willigers, 2016/09/01 14:56:31]

Removed from this patch. I believe the reason is that these have many instances
so having the instrumentation is helpful.

 public:
     InvalidationType type() const { return static_cast<InvalidationType>(m_type); }
     bool isDescendantInvalidationSet() const { return type() == InvalidateDescendants; }