Commit d1a83017 authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Commit Bot

Stop using VisiblePosition in DocumentMarkerController

This patch changes DocumentMarkerController to use VisiblePosition-
free versions of Start/EndOfWord, to reduce usage of VisiblePosition.

Bug: 657237
Change-Id: I344c98c9163ddc06f338e309a6b3e372d5fc2dda
Reviewed-on: https://chromium-review.googlesource.com/c/1330747
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608854}
parent eaf5778e
...@@ -397,13 +397,11 @@ DocumentMarker* DocumentMarkerController::FirstMarkerAroundPosition( ...@@ -397,13 +397,11 @@ DocumentMarker* DocumentMarkerController::FirstMarkerAroundPosition(
return nullptr; return nullptr;
const PositionInFlatTree start_of_word_or_null = const PositionInFlatTree start_of_word_or_null =
StartOfWord(CreateVisiblePosition(position), kPreviousWordIfOnBoundary) StartOfWordPosition(position, kPreviousWordIfOnBoundary);
.DeepEquivalent();
const PositionInFlatTree start = const PositionInFlatTree start =
start_of_word_or_null.IsNotNull() ? start_of_word_or_null : position; start_of_word_or_null.IsNotNull() ? start_of_word_or_null : position;
const PositionInFlatTree end_of_word_or_null = const PositionInFlatTree end_of_word_or_null =
EndOfWord(CreateVisiblePosition(position), kNextWordIfOnBoundary) EndOfWordPosition(position, kNextWordIfOnBoundary);
.DeepEquivalent();
const PositionInFlatTree end = const PositionInFlatTree end =
end_of_word_or_null.IsNotNull() ? end_of_word_or_null : position; end_of_word_or_null.IsNotNull() ? end_of_word_or_null : position;
......
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