Commit e5c72bb7 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Add DCHECK() to NextWordPositionInternal() not to accept null-position

Because of |NextWordPositionInternal()| doesn't accept null-position, this
patch adds |DCHECK()| to catch such callers for ease of debugging.

Bug: 832061
Change-Id: Id987553c939db8f061e58eca09d7ebcd946482c9
Reviewed-on: https://chromium-review.googlesource.com/1013642Reviewed-by: default avatarXiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551531}
parent e0832599
......@@ -81,6 +81,7 @@ PositionTemplate<Strategy> EndOfWordAlgorithm(
PositionInFlatTree NextWordPositionInternal(
const PositionInFlatTree& position) {
DCHECK(position.IsNotNull());
PositionInFlatTree block_end_position;
for (const LayoutBlock* block =
&TextOffsetMapping::ComputeContainigBlock(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