Remove the forced layout in getComputedStyle for elements in Shadow DOM

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

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Neither the name of Google Inc. nor the names of its
@@ skipping 20 matching lines @@
 #include "core/dom/Element.h"
 #include "core/dom/shadow/InsertionPoint.h"
 
 namespace blink {
 
 class LayoutObject;
 
 namespace LayoutTreeBuilderTraversal {
 
 class ParentDetails {
     STACK_ALLOCATED();

[rune, 2016/02/01 09:24:23]

Is this also WTF_MAKE_NONCOPYABLE? Not that I think it needs to be, just to
express the current use pattern?

 public:
-    ParentDetails()
-        : m_insertionPoint(nullptr)
-    { }
+    bool wasDistributedToInsertionPoint() const { return m_wasDistributedToInsertionPoint; }
+    void didTraverseInsertionPoint() { m_wasDistributedToInsertionPoint = true; }
 
-    const InsertionPoint* insertionPoint() const { return m_insertionPoint; }
+    const ShadowRoot* shadowRoot() const { return m_shadowRoot; }
+    void didTraverseShadowRoot(const ShadowRoot* shadowRoot) { m_shadowRoot = shadowRoot; }
 
-    void didTraverseInsertionPoint(const InsertionPoint*);
-
-    bool operator==(const ParentDetails& other)

[rune, 2016/02/01 09:24:23]

Are objects of this class ever being compared? Does it make sense to make
operator== delete?

+    void reset()
     {
-        return m_insertionPoint == other.m_insertionPoint;
+        m_wasDistributedToInsertionPoint = false;
+        m_shadowRoot = nullptr;
     }
 
 private:
-    RawPtrWillBeMember<const InsertionPoint> m_insertionPoint;
+    bool m_wasDistributedToInsertionPoint = false;
+    RawPtrWillBeMember<const ShadowRoot> m_shadowRoot = nullptr;
 };
 
 CORE_EXPORT ContainerNode* parent(const Node&, ParentDetails* = 0);
 CORE_EXPORT Node* firstChild(const Node&);
 CORE_EXPORT Node* nextSibling(const Node&);
 Node* previousSibling(const Node&);
 Node* previous(const Node&, const Node* stayWithin);
 Node* next(const Node&, const Node* stayWithin);
 Node* nextSkippingChildren(const Node&, const Node* stayWithin);
 LayoutObject* nextSiblingLayoutObject(const Node&);
 LayoutObject* previousSiblingLayoutObject(const Node&);
 LayoutObject* nextInTopLayer(const Element&);
 
 inline Element* parentElement(const Node& node)
 {
     ContainerNode* found = parent(node);
     return found && found->isElementNode() ? toElement(found) : 0;
 }
 
 } // namespace LayoutTreeBuilderTraversal
 
 } // namespace blink
 
 #endif