Commit 097ed8e0 authored by Aleks Totic's avatar Aleks Totic Committed by Commit Bot

[TablesNG] Minor fix to element_inner_text

I am uploading minor fixes in parallel.

Change-Id: If4ff1a5047f4b652249c83cba28073a7a9e30b1f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2276707
Commit-Queue: Aleks Totic <atotic@chromium.org>
Reviewed-by: default avatarMorten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#784241}
parent 349a96f7
...@@ -152,12 +152,12 @@ bool ElementInnerTextCollector::IsDisplayBlockLevel(const Node& node) { ...@@ -152,12 +152,12 @@ bool ElementInnerTextCollector::IsDisplayBlockLevel(const Node& node) {
const LayoutObject* const layout_object = node.GetLayoutObject(); const LayoutObject* const layout_object = node.GetLayoutObject();
if (!layout_object) if (!layout_object)
return false; return false;
if (!layout_object->IsLayoutBlock()) {
if (layout_object->IsTableSection()) { if (layout_object->IsTableSection()) {
// Note: |LayoutTableSeleciton::IsInline()| returns false, but it is not // Note: |LayoutTableSection::IsInline()| returns false, but it is not
// block-level. // block-level.
return false; return false;
} }
if (!layout_object->IsLayoutBlock()) {
// Note: Block-level replaced elements, e.g. <img style=display:block>, // Note: Block-level replaced elements, e.g. <img style=display:block>,
// reach here. Unlike |LayoutBlockFlow::AddChild()|, innerText considers // reach here. Unlike |LayoutBlockFlow::AddChild()|, innerText considers
// floats and absolutely-positioned elements as block-level node. // floats and absolutely-positioned elements as block-level node.
......
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