Make SetSelectionToDragCaret() not to use...

Make SetSelectionToDragCaret() not to use FrameSelection::ComputeVisibileSelectionInDOMTreeDeprecated()

This patch changes |SetSelectionToDragCaret()| to use
|FrameSelection::ComputeVisibileSelectionInDOMTree| instead of deprecated
version for improving code health.

This is 2/3 of series of patches:
 1. http://crrev.com/c/This patch
 2. This patch
 3. http://crrev.com/c/1153071

Bug: 698633
Change-Id: Ie51e4b8524c4429fb04a065cb13264e7277d2697
Reviewed-on: https://chromium-review.googlesource.com/1153070
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579296}

third_party/blink/renderer/core/page/drag_controller.cc

@@ -474,29 +474,28 @@ static bool SetSelectionToDragCaret(LocalFrame* frame,
                                     Range*& range,
                                     const LayoutPoint& point) {
   frame->Selection().SetSelectionAndEndTyping(drag_caret);
-  if (frame->Selection()
-          .ComputeVisibleSelectionInDOMTreeDeprecated()
-          .IsNone()) {
-    // TODO(editing-dev): The use of
-    // updateStyleAndLayoutIgnorePendingStylesheets
-    // needs to be audited.  See http://crbug.com/590369 for more details.
-    // |LocalFrame::positinForPoint()| requires clean layout.
-    frame->GetDocument()->UpdateStyleAndLayoutIgnorePendingStylesheets();
+  // TODO(editing-dev): The use of
+  // UpdateStyleAndLayoutIgnorePendingStylesheets
+  // needs to be audited.  See http://crbug.com/590369 for more details.
+  frame->GetDocument()->UpdateStyleAndLayoutIgnorePendingStylesheets();
+  if (frame->Selection().ComputeVisibleSelectionInDOMTree().IsNone()) {
     const PositionWithAffinity& position = frame->PositionForPoint(point);
     if (!position.IsConnected())
       return false;
 
     frame->Selection().SetSelectionAndEndTyping(
         SelectionInDOMTree::Builder().Collapse(position).Build());
+    // TODO(editing-dev): The use of
+    // UpdateStyleAndLayoutIgnorePendingStylesheets
+    // needs to be audited.  See http://crbug.com/590369 for more details.
+    frame->GetDocument()->UpdateStyleAndLayoutIgnorePendingStylesheets();
     const VisibleSelection& drag_caret =
-        frame->Selection().ComputeVisibleSelectionInDOMTreeDeprecated();
+        frame->Selection().ComputeVisibleSelectionInDOMTree();
     range = CreateRange(drag_caret.ToNormalizedEphemeralRange());
   }
-  return !frame->Selection()
-              .ComputeVisibleSelectionInDOMTreeDeprecated()
-              .IsNone() &&
+  return !frame->Selection().ComputeVisibleSelectionInDOMTree().IsNone() &&
          frame->Selection()
-             .ComputeVisibleSelectionInDOMTreeDeprecated()
+             .ComputeVisibleSelectionInDOMTree()
              .IsContentEditable();
 }