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

Make LayoutObject::EnclosingBlockFlowFragment() strict

This patch changes |LayoutObject::EnclosingBlockFlowFragment()| to strict by
using |ChildrenInline()| instead of |HasNGInlineNodeData()| for detecting
failures of layout operation which doesn't compute inline node data.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Bug: 714962
Change-Id: I14e724e06247c59e2f80a8a4db7dc1ef6e5433c7
Reviewed-on: https://chromium-review.googlesource.com/885542Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531854}
parent 4859b2d9
......@@ -746,7 +746,7 @@ LayoutBlockFlow* LayoutObject::EnclosingNGBlockFlow() const {
const NGPhysicalBoxFragment* LayoutObject::EnclosingBlockFlowFragment() const {
DCHECK(IsInline() || IsText());
LayoutBlockFlow* const block_flow = EnclosingNGBlockFlow();
if (!block_flow || !block_flow->HasNGInlineNodeData())
if (!block_flow || !block_flow->ChildrenInline())
return nullptr;
// TODO(kojii): CurrentFragment isn't always available after layout clean.
// Investigate why.
......
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