Commit 1bdae13c authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

Get rid of redundant call of EmitEndBlock() in Element#innerText

This patch gets rid of redundant call of |EmitEndBlock()| after calling
ProcessChildrenWithRequiredLineBreaks() which calls |EmitEndBlock()| at
end of function in |Element#innerText| for improving code health.

Since EmitEndBlock() turns |has_collapsible_space_| to false, second
call has no effect.

Change-Id: I44012380d7335b33f901954ac24a85e9cd4e98b4
Reviewed-on: https://chromium-review.googlesource.com/c/1290058Reviewed-by: default avatarYoshifumi Inoue <yosin@chromium.org>
Reviewed-by: default avatarYoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601088}
parent 4327bb89
......@@ -494,7 +494,6 @@ void ElementInnerTextCollector::ProcessNode(const Node& node) {
const LayoutObject& layout_object = *node.GetLayoutObject();
if (style->Display() == EDisplay::kTableCell) {
ProcessChildrenWithRequiredLineBreaks(node, 0);
result_.EmitEndBlock();
if (layout_object.IsTableCell() &&
ToLayoutTableCell(layout_object).NextCell())
result_.EmitTab();
......
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