Do not call updateDistribution() in Node::assingedSlotForBinding()

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 2282 matching lines @@
   for (const auto& insertionPoint : insertionPoints) {
     DCHECK(insertionPoint->containingShadowRoot());
     if (!insertionPoint->containingShadowRoot()->isOpenOrV0())
       break;
     filteredInsertionPoints.append(insertionPoint);
   }
   return StaticNodeList::adopt(filteredInsertionPoints);
 }
 
 HTMLSlotElement* Node::assignedSlot() const {
+  // assignedSlot doesn't needs to call updateDistribution().

[kochi, 2016/12/09 08:29:04]

s/needs/need/

   DCHECK(!isPseudoElement());
   if (ShadowRoot* root = v1ShadowRootOfParent())
     return root->ensureSlotAssignment().findSlot(*this);
   return nullptr;
 }
 
 HTMLSlotElement* Node::assignedSlotForBinding() {
-  updateDistribution();
+  // assignedSlot doesn't needs to call updateDistribution().

[kochi, 2016/12/09 08:29:04]

Ditto.

   if (ShadowRoot* root = v1ShadowRootOfParent()) {
     if (root->type() == ShadowRootType::Open)
       return root->ensureSlotAssignment().findSlot(*this);
   }
   return nullptr;
 }
 
 void Node::setFocused(bool flag) {
   document().userActionElements().setFocused(this, flag);
 }
@@ skipping 220 matching lines @@
   if (node) {
     std::stringstream stream;
     node->printNodePathTo(stream);
     LOG(INFO) << stream.str();
   } else {
     LOG(INFO) << "Cannot showNodePath for <null>";
   }
 }
 
 #endif