Commit 1ac52e75 authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Commit Bot

Use uncanonicalized EndOfWordPosition() in visible selection end adjuster

This patch hoists the creation of VisiblePosition from EndOfWord
to its caller ComputeEndRespectingGranularityAlgorithm() to reduce
the usage VisiblePosition.

Bug: 657237
Change-Id: I29cc408d35ecc30db4aa635f5ad4bed88c0032c5
Reviewed-on: https://chromium-review.googlesource.com/c/1330789Reviewed-by: default avatarYoichi Osato <yoichio@chromium.org>
Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608026}
parent fc1e38cd
...@@ -186,7 +186,8 @@ class GranularityAdjuster final { ...@@ -186,7 +186,8 @@ class GranularityAdjuster final {
const VisiblePositionTemplate<Strategy> original_end = const VisiblePositionTemplate<Strategy> original_end =
CreateVisiblePosition(passed_end); CreateVisiblePosition(passed_end);
const VisiblePositionTemplate<Strategy> word_end = const VisiblePositionTemplate<Strategy> word_end =
EndOfWord(original_end, ChooseWordSide(original_end)); CreateVisiblePosition(EndOfWordPosition(
passed_end.GetPosition(), ChooseWordSide(original_end)));
if (!IsEndOfParagraph(original_end)) if (!IsEndOfParagraph(original_end))
return word_end.DeepEquivalent(); return word_end.DeepEquivalent();
if (IsEmptyTableCell(start.AnchorNode())) if (IsEmptyTableCell(start.AnchorNode()))
......
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