Commit 18322439 authored by yosin's avatar yosin Committed by Commit bot

Get rid of an unused parameter in FrameSelection::moveTo()

This patch gets rid of unused parameter in |Position| version of
|FrameSelection::moveTo()| to simplify code for improving code health.

BUG=n/a
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2021983002
Cr-Commit-Position: refs/heads/master@{#396784}
parent ed9fa2dc
......@@ -175,9 +175,9 @@ void FrameSelection::moveTo(const VisiblePosition &base, const VisiblePosition &
setSelection(VisibleSelection(base, extent, selectionHasDirection), options);
}
void FrameSelection::moveTo(const Position &pos, TextAffinity affinity, EUserTriggered userTriggered)
void FrameSelection::moveTo(const Position &pos, TextAffinity affinity)
{
SetSelectionOptions options = CloseTyping | ClearTypingStyle | userTriggered;
SetSelectionOptions options = CloseTyping | ClearTypingStyle;
setSelection(VisibleSelection(pos, affinity, selection().isDirectional()), options);
}
......
......@@ -104,7 +104,7 @@ public:
void moveTo(const VisiblePosition&, EUserTriggered = NotUserTriggered, CursorAlignOnScroll = CursorAlignOnScroll::IfNeeded);
void moveTo(const VisiblePosition&, const VisiblePosition&, EUserTriggered = NotUserTriggered);
void moveTo(const Position&, TextAffinity, EUserTriggered = NotUserTriggered);
void moveTo(const Position&, TextAffinity);
template <typename Strategy>
const VisibleSelectionTemplate<Strategy>& visibleSelection() const;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment