Re-order Node flags based on semantics
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 17 Sep 2018 21:07:08 +0000 (21:07 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 17 Sep 2018 21:07:08 +0000 (21:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189643

Reviewed by Simon Fraser.

Re-ordered Node flags based on their category and subclasses of Node which use them.

* dom/Node.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236083 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/Node.h

index c19c24d..00d46b9 100644 (file)
@@ -1,3 +1,14 @@
+2018-09-14  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Re-order Node flags based on semantics
+        https://bugs.webkit.org/show_bug.cgi?id=189643
+
+        Reviewed by Simon Fraser.
+
+        Re-ordered Node flags based on their category and subclasses of Node which use them.
+
+        * dom/Node.h:
+
 2018-09-17  Simon Fraser  <simon.fraser@apple.com>
 
         Add more ResourceLoading logging, particularly in MemoryCache code
index f70f4f3..27de22e 100644 (file)
@@ -559,39 +559,41 @@ protected:
         IsHTMLFlag = 1 << 3,
         IsSVGFlag = 1 << 4,
         IsMathMLFlag = 1 << 5,
-        IsConnectedFlag = 1 << 6,
-        IsInShadowTreeFlag = 1 << 7,
-        IsDocumentNodeFlag = 1 << 8,
-        IsShadowRootFlag = 1 << 9,
+        IsDocumentNodeFlag = 1 << 6,
+        IsShadowRootFlag = 1 << 7,
+        IsConnectedFlag = 1 << 8,
+        IsInShadowTreeFlag = 1 << 9,
         HasRareDataFlag = 1 << 10,
         HasEventTargetDataFlag = 1 << 11,
 
         // These bits are used by derived classes, pulled up here so they can
         // be stored in the same memory word as the Node bits above.
         ChildNeedsStyleRecalcFlag = 1 << 12, // ContainerNode
-
-        IsParsingChildrenFinishedFlag = 1 << 13, // Element
-        StyleValidityShift = 14,
+        DirectChildNeedsStyleRecalcFlag = 1 << 13,
+
+        IsEditingTextOrUndefinedCustomElementFlag = 1 << 14, // Text and Element
+        IsCustomElement = 1 << 15, // Element
+        HasFocusWithin = 1 << 16,
+        IsLinkFlag = 1 << 17,
+        IsUserActionElement = 1 << 18,
+        IsParsingChildrenFinishedFlag = 1 << 19,
+        HasSyntheticAttrChildNodesFlag = 1 << 20,
+        SelfOrAncestorHasDirAutoFlag = 1 << 21,
+
+        // The following flags are used in style invalidation.
+        StyleValidityShift = 22,
         StyleValidityMask = 3 << StyleValidityShift,
-        StyleResolutionShouldRecompositeLayerFlag = 1 << 16,
-        IsEditingTextOrUndefinedCustomElementFlag = 1 << 17,
-        HasFocusWithin = 1 << 18,
-        HasSyntheticAttrChildNodesFlag = 1 << 19,
-        HasCustomStyleResolveCallbacksFlag = 1 << 20,
-        DescendantsAffectedByPreviousSiblingFlag = 1 << 21,
-        IsCustomElement = 1 << 22,
-        IsLinkFlag = 1 << 23,
-        IsUserActionElement = 1 << 24,
+        StyleResolutionShouldRecompositeLayerFlag = 1 << 24,
 
         ChildrenAffectedByFirstChildRulesFlag = 1 << 25,
         ChildrenAffectedByLastChildRulesFlag = 1 << 26,
         ChildrenAffectedByHoverRulesFlag = 1 << 27,
 
-        DirectChildNeedsStyleRecalcFlag = 1 << 28,
-        AffectsNextSiblingElementStyle = 1 << 29,
-        StyleIsAffectedByPreviousSibling = 1 << 30,
+        AffectsNextSiblingElementStyle = 1 << 28,
+        StyleIsAffectedByPreviousSibling = 1 << 29,
+        DescendantsAffectedByPreviousSiblingFlag = 1 << 30,
 
-        SelfOrAncestorHasDirAutoFlag = 1 << 31,
+        HasCustomStyleResolveCallbacksFlag = 1 << 31,
 
         DefaultNodeFlags = IsParsingChildrenFinishedFlag
     };