DescriptionIntroduce Find{Left,Right}BidiRun() and utilize in AdjustInlineBoxPositionForTextDirection()
This patch introduces |Find{Left,Right}BidiRun()| in |InlineBoxTraversal| to
unify common pattern into function for improving of readability.
Before this patch:
InlineBox* result = start;
do {
result = result->Forward();
} while (result && result->BidiLevel() > level);
... use result ...
After this patch:
InlineBox* Find(InlineBox* start, unsigned level) {
for (InlineBox* runner = start->Forward(); runner;
runner = runner->Forward()) {
if (runner->BidiLevel() <= level)
return runner;
}
return nullptr;
}
InlineBox* result = Find(start, level);
... use result ...
BUG=707656
TEST=n/a; no behavior changes
Review-Url: https://codereview.chromium.org/2943053002
Cr-Commit-Position: refs/heads/master@{#480340}
Committed: https://chromium.googlesource.com/chromium/src/+/add3c68a6c8aea99a4f7088193f99d5eaaa5b510
Patch Set 1 : 2017-06-16T13:52:56 #
Messages
Total messages: 22 (16 generated)
|