Use EphemeralRange instead of Range in Range::isNodeFullyContainded

third_party/WebKit/Source/core/input/MouseEventManager.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/input/MouseEventManager.h"
 
 #include "core/clipboard/DataObject.h"
 #include "core/clipboard/DataTransfer.h"
 #include "core/dom/Element.h"
 #include "core/dom/ElementTraversal.h"
@@ skipping 413 matching lines @@
 
   // To fix <rdar://problem/4895428> Can't drag selected ToDo, we don't focus
   // a node on mouse down if it's selected and inside a focused node. It will
   // be focused if the user does a mouseup over it, however, because the
   // mouseup will set a selection inside it, which will call
   // FrameSelection::setFocusedNodeIfNeeded.
   if (element &&
       m_frame->selection()
           .computeVisibleSelectionInDOMTreeDeprecated()
           .isRange()) {
-    // TODO(yosin) We should not create |Range| object for calling
-    // |isNodeFullyContained()|.
-    if (createRange(m_frame->selection()
-                        .computeVisibleSelectionInDOMTreeDeprecated()
-                        .toNormalizedEphemeralRange())
-            ->isNodeFullyContained(*element) &&
+    const EphemeralRange& range =
+        m_frame->selection()
+            .computeVisibleSelectionInDOMTreeDeprecated()
+            .toNormalizedEphemeralRange();
+    if (isNodeFullyContained(range, *element) &&
         element->isDescendantOf(m_frame->document()->focusedElement()))
       return WebInputEventResult::NotHandled;
   }
 
   // Only change the focus when clicking scrollbars if it can transfered to a
   // mouse focusable node.
   if (!element && hitTestResult.scrollbar())
     return WebInputEventResult::HandledSystem;
 
   if (Page* page = m_frame->page()) {
@@ skipping 572 matching lines @@
 
 void MouseEventManager::setClickCount(int clickCount) {
   m_clickCount = clickCount;
 }
 
 bool MouseEventManager::mouseDownMayStartDrag() {
   return m_mouseDownMayStartDrag;
 }
 
 }  // namespace blink