Store nonAttachedStyle on NodeLayoutData instead of on HashMap in Document.

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

 /*
  * 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, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  *
@@ skipping 587 matching lines @@
   if (!node_layout_data->IsSharedEmptyData()) {
     node_layout_data->SetLayoutObject(layout_object);
     return;
   }
 
   if (!layout_object)
     return;
 
   // Swap the NodeLayoutData to point to a new NodeLayoutData instead of the
   // static SharedEmptyData instance.
-  node_layout_data = new NodeLayoutData(layout_object);
+  DCHECK(!node_layout_data->GetNonAttachedStyle());
+  node_layout_data = new NodeLayoutData(layout_object, nullptr);
   if (HasRareData())

[nainar, 2017/04/19 01:38:34]

This can read as below instead:

  DCHECK(!node_layout_data->GetNonAttachedStyle());
  // Assign a nullptr to everything
  node_layout_data =
ensureMutableNodeLayoutData()->SetLayoutObject(layout_object);

In ensureMutableNodeLayoutData() -> 
  NodeLayoutData* data = new NodeLayoutData();
  if (HasRareData())
    data_.rare_data_->SetNodeLayoutData(data);
  else
    data_.node_layout_data_ = data;
  return data;

     data_.rare_data_->SetNodeLayoutData(node_layout_data);
   else
+    data_.node_layout_data_ = node_layout_data;
+}
+
+void Node::SetNonAttachedStyle(RefPtr<ComputedStyle> non_attached_style) {
+  NodeLayoutData* node_layout_data = HasRareData()

[nainar, 2017/04/19 01:38:34]

I can pull out this into a getter

+                                         ? data_.rare_data_->GetNodeLayoutData()
+                                         : data_.node_layout_data_;
+
+  // Already pointing to a non empty NodeLayoutData so just set the pointer to
+  // the new LayoutObject.
+  if (!node_layout_data->IsSharedEmptyData()) {
+    node_layout_data->SetNonAttachedStyle(non_attached_style);
+    return;
+  }
+
+  if (!non_attached_style)
+    return;
+
+  // Swap the NodeLayoutData to point to a new NodeLayoutData instead of the
+  // static SharedEmptyData instance.
+  DCHECK(!node_layout_data->GetLayoutObject());
+  node_layout_data = new NodeLayoutData(nullptr, non_attached_style);
+  if (HasRareData())
+    data_.rare_data_->SetNodeLayoutData(node_layout_data);
+  else
     data_.node_layout_data_ = node_layout_data;
 }
 
 LayoutBoxModelObject* Node::GetLayoutBoxModelObject() const {
   LayoutObject* layout_object = this->GetLayoutObject();
   return layout_object && layout_object->IsBoxModelObject()
              ? ToLayoutBoxModelObject(layout_object)
              : nullptr;
 }
 
@@ skipping 1941 matching lines @@
   if (node) {
     std::stringstream stream;
     node->PrintNodePathTo(stream);
     LOG(INFO) << stream.str();
   } else {
     LOG(INFO) << "Cannot showNodePath for <null>";
   }
 }
 
 #endif