Move LayoutObject to satellite NodeLayoutData that hangs from a Node.

third_party/WebKit/Source/core/dom/Node.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004-2011, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 79 matching lines @@
   kFailed = 3 << kNodeCustomElementShift,
 
   kNotDefinedFlag = 2 << kNodeCustomElementShift,
 };
 
 enum class SlotChangeType {
   kInitial,
   kChained,
 };
 
-class NodeRareDataBase {
+class LayoutData {

[esprehn, 2017/04/10 22:20:12]

NodeLayoutData maybe? LayoutData is pretty generic, it's also a bit unfortunate
that this thing is both style and layout, I miss the days when we just called
this stuff Rendering :P But the bike shed is not super important.

[nainar, 2017/04/13 00:10:22]

Done.

  public:
+  LayoutData(LayoutObject* layout_object) : layout_object_(layout_object) {}

[esprehn, 2017/04/10 22:20:12]

explicit on all single arg constructors unless you want to allow implicit
construction, you probably don't want that here.

[nainar, 2017/04/13 00:10:22]

Done.

   LayoutObject* GetLayoutObject() const { return layout_object_; }
   void SetLayoutObject(LayoutObject* layout_object) {
     layout_object_ = layout_object;
   }
 
- protected:
-  NodeRareDataBase(LayoutObject* layout_object)
-      : layout_object_(layout_object) {}
-
- protected:
-  // LayoutObjects are fully owned by their DOM node. See LayoutObject's
-  // LIFETIME documentation section.
+ private:
   LayoutObject* layout_object_;
 };
 
+class NodeRareDataBase {
+ public:
+  LayoutData* GetLayoutData() const { return layout_data_; }
+  void SetLayoutData(LayoutData* layout_data) { layout_data_ = layout_data; }
+
+ protected:
+  NodeRareDataBase(LayoutObject* layout_object) {
+    layout_data_ = new LayoutData(layout_object);
+  }
+
+ protected:
+  LayoutData* layout_data_;
+};
+
 class Node;
 WILL_NOT_BE_EAGERLY_TRACED_CLASS(Node);
 
 // This class represents a DOM node in the DOM tree.
 // https://dom.spec.whatwg.org/#interface-node
 class CORE_EXPORT Node : public EventTarget {
   DEFINE_WRAPPERTYPEINFO();
   friend class TreeScope;
   friend class TreeScopeAdopter;
 
@@ skipping 429 matching lines @@
   virtual bool CanStartSelection() const;
 
   // -----------------------------------------------------------------------------
   // Integration with layout tree
 
   // As layoutObject() includes a branch you should avoid calling it repeatedly
   // in hot code paths.
   // Note that if a Node has a layoutObject, it's parentNode is guaranteed to
   // have one as well.
   LayoutObject* GetLayoutObject() const {
-    return HasRareData() ? data_.rare_data_->GetLayoutObject()
-                         : data_.layout_object_;
+    return HasRareData() ? data_.rare_data_->GetLayoutData()->GetLayoutObject()
+                         : data_.layout_data_->GetLayoutObject();
   }
   void SetLayoutObject(LayoutObject* layout_object) {
     if (HasRareData())
-      data_.rare_data_->SetLayoutObject(layout_object);
+      data_.rare_data_->GetLayoutData()->SetLayoutObject(layout_object);
     else
-      data_.layout_object_ = layout_object;
+      data_.layout_data_->SetLayoutObject(layout_object);
   }
 
   // Use these two methods with caution.
   LayoutBox* GetLayoutBox() const;
   LayoutBoxModelObject* GetLayoutBoxModelObject() const;
 
   struct AttachContext {
     STACK_ALLOCATED();
     ComputedStyle* resolved_style = nullptr;
     bool performing_reattach = false;
@@ skipping 334 matching lines @@
   const HeapHashSet<TraceWrapperMember<MutationObserverRegistration>>*
   TransientMutationObserverRegistry();
 
   uint32_t node_flags_;
   Member<ContainerNode> parent_or_shadow_host_node_;
   Member<TreeScope> tree_scope_;
   Member<Node> previous_;
   Member<Node> next_;
   // When a node has rare data we move the layoutObject into the rare data.
   union DataUnion {
-    DataUnion() : layout_object_(nullptr) {}
+    DataUnion() : layout_data_(nullptr) {}
     // LayoutObjects are fully owned by their DOM node. See LayoutObject's
     // LIFETIME documentation section.
-    LayoutObject* layout_object_;
+    LayoutData* layout_data_;
     NodeRareDataBase* rare_data_;
   } data_;
 };
 
 inline void Node::SetParentOrShadowHostNode(ContainerNode* parent) {
   DCHECK(IsMainThread());
   parent_or_shadow_host_node_ = parent;
   ScriptWrappableVisitor::WriteBarrier(
       this, reinterpret_cast<Node*>(parent_or_shadow_host_node_.Get()));
 }
@@ skipping 52 matching lines @@
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showNode(const blink::Node*);
 void showTree(const blink::Node*);
 void showNodePath(const blink::Node*);
 #endif
 
 #endif  // Node_h