Prefer ComposedTreeTraversal to NodeRenderingTraversal.

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/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 29 matching lines @@
 #include "core/dom/Document.h"
 #include "core/dom/DocumentFragment.h"
 #include "core/dom/DocumentMarkerController.h"
 #include "core/dom/DocumentType.h"
 #include "core/dom/Element.h"
 #include "core/dom/ElementRareData.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/LiveNodeList.h"
 #include "core/dom/NodeRareData.h"
-#include "core/dom/NodeRenderingTraversal.h"
 #include "core/dom/NodeTraversal.h"
 #include "core/dom/ProcessingInstruction.h"
 #include "core/dom/Range.h"
 #include "core/dom/StaticNodeList.h"
 #include "core/dom/TemplateContentDocumentFragment.h"
 #include "core/dom/Text.h"
 #include "core/dom/TreeScopeAdopter.h"
 #include "core/dom/UserActionElementSet.h"
 #include "core/dom/WeakNodeMap.h"
 #include "core/dom/shadow/ComposedTreeTraversal.h"
@@ skipping 727 matching lines @@
 
 bool Node::shouldHaveFocusAppearance() const
 {
     ASSERT(focused());
     return true;
 }
 
 bool Node::isInert() const
 {
     const HTMLDialogElement* dialog = document().activeModalDialog();
-    if (dialog && this != document() && !NodeRenderingTraversal::contains(*dialog, *this))
+    if (dialog && this != document() && canParticipateInComposedTree() && !ComposedTreeTraversal::contains(*dialog, *this))

[kochi, 2015/04/09 05:01:47]

(Just a comment)
This looks somewhat tricky as other parts are mechanical replacement, while
this CTT::contains() you have to take care that the target element should meet
canParticipateInComposedTree() condition.
It should be okay because that condition is caugut in CTT::contains() as
assertPrecondition().

[hayato, 2015/04/09 05:35:01]

Acknowledged. 
canParticipateInComposedTree() is necessary here to avoid hitting assertion
because Node::isInert might be called for (active) insertion points.

         return true;
     return document().ownerElement() && document().ownerElement()->isInert();
 }
 
 unsigned Node::nodeIndex() const
 {
     Node *_tempNode = previousSibling();
     unsigned count = 0;
     for (count = 0; _tempNode; count++)
         _tempNode = _tempNode->previousSibling();
@@ skipping 1004 matching lines @@
         rootNode = parentOrShadowHostOrFrameOwner(rootNode);
     showSubTreeAcrossFrame(rootNode, this, "");
 }
 
 #endif
 
 // --------
 
 Element* Node::enclosingLinkEventParentOrSelf()
 {
-    for (Node* node = this; node; node = NodeRenderingTraversal::parent(*node)) {
+    for (Node* node = this; node; node = ComposedTreeTraversal::parent(*node)) {
         // For imagemaps, the enclosing link node is the associated area element not the image itself.
         // So we don't let images be the enclosingLinkNode, even though isLink sometimes returns true
         // for them.
         if (node->isLink() && !isHTMLImageElement(*node)) {
             // Casting to Element is safe because only HTMLAnchorElement, HTMLImageElement and
             // SVGAElement can return true for isLink().
             return toElement(node);
         }
     }
 
@@ skipping 672 matching lines @@
         node->showTreeForThis();
 }
 
 void showNodePath(const blink::Node* node)
 {
     if (node)
         node->showNodePathForThis();
 }
 
 #endif