Commit 07fb5bbe authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Commit Bot

Remove an unnecessary if statement from InlineBoxPosition.cpp

The |if| statement's condition is always false, since |level| is set to
|inline_box->BidiLevel()| in L165.

Note: The code was originally introduced in crrev.com/edfe20, which
already contains this redundant branching.

Bug: 771398
Change-Id: If83f178b8ab91f15fcc7176e429541e69a681b15
Reviewed-on: https://chromium-review.googlesource.com/991604Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547762}
parent 9e0153d9
......@@ -183,11 +183,8 @@ InlineBoxPosition AdjustInlineBoxPositionForTextDirectionInternal(
return InlineBoxPosition(result_box, result_box->CaretLeftmostOffset());
}
if (unicode_bidi == UnicodeBidi::kPlaintext) {
if (inline_box->BidiLevel() < level)
return InlineBoxPosition(inline_box, inline_box->CaretLeftmostOffset());
if (unicode_bidi == UnicodeBidi::kPlaintext)
return InlineBoxPosition(inline_box, inline_box->CaretRightmostOffset());
}
InlineBox* const next_box = inline_box->NextLeafChildIgnoringLineBreak();
if (!next_box || next_box->BidiLevel() < level) {
......
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