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);
+  node_layout_data = new NodeLayoutData(
+      layout_object, node_layout_data->GetNonAttachedStyle());

[nainar, 2017/04/18 03:48:12]

at this stage the GetNonAttachedStyle call should return a nullptr. Assign
nullptr here instead?

[esprehn, 2017/04/18 21:31:10]

Yeah can we make this a DCHECK and remove the constructor argument instead? Just
initialize directly to nullptr. I don't think saving the one assignment is worth
it in general.

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

Done.

   if (HasRareData())
     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()
+                                         ? 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.
+  node_layout_data = new NodeLayoutData(node_layout_data->GetLayoutObject(),

[nainar, 2017/04/18 03:48:12]

at this stage the GetLayoutObject call should return a nullptr. Assign nullptr
here instead?

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

Done. 

+                                        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