Make GranularityStrategy::updateExtent() to return SelectionInDOM instead of VisibleSelection

third_party/WebKit/Source/core/editing/FrameSelection.cpp

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1074 matching lines @@
     m_granularityStrategy = WTF::makeUnique<DirectionGranularityStrategy>();
   else
     m_granularityStrategy = WTF::makeUnique<CharacterGranularityStrategy>();
   return m_granularityStrategy.get();
 }
 
 void FrameSelection::moveRangeSelectionExtent(const IntPoint& contentsPoint) {
   if (computeVisibleSelectionInDOMTreeDeprecated().isNone())
     return;
 
-  VisibleSelection newSelection =
-      granularityStrategy()->updateExtent(contentsPoint, m_frame);
-  setSelection(SelectionInDOMTree::Builder(newSelection.asSelection())
+  const SetSelectionOptions options =
+      FrameSelection::CloseTyping | FrameSelection::ClearTypingStyle |
+      FrameSelection::DoNotClearStrategy | UserTriggered;
+  setSelection(SelectionInDOMTree::Builder(
+                   granularityStrategy()->updateExtent(contentsPoint, m_frame))
                    .setIsHandleVisible(true)
                    .build(),
-               FrameSelection::CloseTyping | FrameSelection::ClearTypingStyle |
-                   FrameSelection::DoNotClearStrategy | UserTriggered,
-               CursorAlignOnScroll::IfNeeded, CharacterGranularity);
+               options);
 }
 
 // TODO(yosin): We should make |FrameSelection::moveRangeSelection()| to take
 // two |IntPoint| instead of two |VisiblePosition| like
 // |moveRangeSelectionExtent()|.
 void FrameSelection::moveRangeSelection(const VisiblePosition& basePosition,
                                         const VisiblePosition& extentPosition,
                                         TextGranularity granularity) {
   SelectionInDOMTree newSelection =
       SelectionInDOMTree::Builder()
@@ skipping 45 matching lines @@
 }
 
 void showTree(const blink::FrameSelection* sel) {
   if (sel)
     sel->showTreeForThis();
   else
     LOG(INFO) << "Cannot showTree for <null> FrameSelection.";
 }
 
 #endif