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

Add a comment to LineLayoutInlineItem::SetAncestorLineBoxDirty()

This patch adds a comment to |LineLayoutInlineItem::SetAncestorLineBoxDirty()|.

Change-Id: I504e7a12494e92752c7ab60916b6549e523c870d
Reviewed-on: https://chromium-review.googlesource.com/773455Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517057}
parent 02dfc196
......@@ -215,8 +215,11 @@ class LineLayoutItem {
bool SelfNeedsLayout() const { return layout_object_->SelfNeedsLayout(); }
// TODO(dgrogan/eae): Why does layoutObject need to know if its ancestor
// line box is dirty at all?
// |SetAncestorLineBoxDirty()| invalidates |layout_object|, should be
// |LayoutInline|, with |kLineBoxesChanged|.
// Note: |AncestorLineBoxDirty| flag itself is used for preventing
// invalidation on |layout_object_| more than once and used only in
// |LineBoxList::DirtyLinesFromChangedChild()|.
void SetAncestorLineBoxDirty() const {
layout_object_->SetAncestorLineBoxDirty();
}
......
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