[LayoutNG] Introduce block child iterator.

third_party/WebKit/Source/core/layout/ng/ng_break_token.h

Index: third_party/WebKit/Source/core/layout/ng/ng_break_token.h
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_break_token.h b/third_party/WebKit/Source/core/layout/ng/ng_break_token.h
index 15f0b1e0399a080538559f3e1994f9b871792f81..191ef8ce932672f16a3cfbb5e8c5256500e1c09a 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_break_token.h
+++ b/third_party/WebKit/Source/core/layout/ng/ng_break_token.h
@@ -39,8 +39,10 @@ class CORE_EXPORT NGBreakToken
   enum NGBreakTokenType { kBlockBreakToken, kTextBreakToken };
   NGBreakTokenType Type() const { return static_cast<NGBreakTokenType>(type_); }
 
+  enum NGBreakTokenStatus { kUnFinished, kFinished };

[mstensho (USE GERRIT), 2017/02/22 20:39:19]

Nit: Since it's spelled "unfinished", and not "un-finished", I think kUnfinished
is the right name.

[ikilpatrick, 2017/02/22 22:25:22]

Done.

+
   // Whether the layout node cannot produce any more fragments.
-  bool IsFinished() const { return is_finished_; }
+  bool IsFinished() const { return status_ == kFinished; }
 
   // Returns the node associated with this break token. A break token cannot be
   // used with any other node.
@@ -49,12 +51,14 @@ class CORE_EXPORT NGBreakToken
   DEFINE_INLINE_VIRTUAL_TRACE() { visitor->trace(node_); }
 
  protected:
-  NGBreakToken(NGBreakTokenType type, bool is_finished, NGLayoutInputNode* node)
-      : type_(type), is_finished_(is_finished), node_(node) {}
+  NGBreakToken(NGBreakTokenType type,
+               NGBreakTokenStatus status,
+               NGLayoutInputNode* node)
+      : type_(type), status_(status), node_(node) {}
 
  private:
   unsigned type_ : 1;
-  unsigned is_finished_ : 1;
+  unsigned status_ : 1;
 
   Member<NGLayoutInputNode> node_;
 };