Expand FrameSeleciton::end() to increase chances of hoisting update layout

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

 // Copyright 2015 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/EventHandler.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/Range.h"
 #include "core/editing/Editor.h"
 #include "core/editing/FrameSelection.h"
@@ skipping 145 matching lines @@
   ASSERT_TRUE(selection().isCaret());
   EXPECT_EQ(Position(line, 0), selection().start());
 
   // Multi-tap events on editable elements should trigger selection, just
   // like multi-click events.
   TapEventBuilder doubleTapEvent(IntPoint(0, 0), 2);
   document().frame()->eventHandler().handleGestureEvent(doubleTapEvent);
   ASSERT_TRUE(selection().isRange());
   EXPECT_EQ(Position(line, 0), selection().start());
   if (document().frame()->editor().isSelectTrailingWhitespaceEnabled()) {
-    EXPECT_EQ(Position(line, 4), selection().end());
+    EXPECT_EQ(Position(line, 4),
+              selection().computeVisibleSelectionInDOMTreeDeprecated().end());
     EXPECT_EQ("One ", WebString(selection().selectedText()).utf8());
   } else {
-    EXPECT_EQ(Position(line, 3), selection().end());
+    EXPECT_EQ(Position(line, 3),
+              selection().computeVisibleSelectionInDOMTreeDeprecated().end());
     EXPECT_EQ("One", WebString(selection().selectedText()).utf8());
   }
 
   TapEventBuilder tripleTapEvent(IntPoint(0, 0), 3);
   document().frame()->eventHandler().handleGestureEvent(tripleTapEvent);
   ASSERT_TRUE(selection().isRange());
   EXPECT_EQ(Position(line, 0), selection().start());
-  EXPECT_EQ(Position(line, 13), selection().end());
+  EXPECT_EQ(Position(line, 13),
+            selection().computeVisibleSelectionInDOMTreeDeprecated().end());
   EXPECT_EQ("One Two Three", WebString(selection().selectedText()).utf8());
 }
 
 TEST_F(EventHandlerTest, multiClickSelectionFromTapDisabledIfNotEditable) {
   setHtmlInnerHTML(
       "<style> body { margin: 0px; } .line { display: block; width: 300px; "
       "height: 30px; } </style>"
       "<span class='line' id='line'>One Two Three</span>");
 
   Node* line = document().getElementById("line")->firstChild();
@@ skipping 241 matching lines @@
       WebPointerProperties::Button::Left, 1, WebInputEvent::NoModifiers,
       TimeTicks::Now().InSeconds());
   mouseUpEvent.setFrameScale(1);
   document().frame()->eventHandler().dragSourceEndedAt(mouseUpEvent,
                                                        DragOperationNone);
 
   // This test passes if it doesn't crash.
 }
 
 }  // namespace blink