[LayoutNG] Initial implementation of NGBoxIterator

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NGBlockLayoutAlgorithm_h
 #define NGBlockLayoutAlgorithm_h
 
 #include "core/layout/ng/ng_box.h"
+#include "core/layout/ng/ng_box_iterator.h"
 #include "wtf/RefPtr.h"
 
 namespace blink {
 
 class ComputedStyle;
 class NGConstraintSpace;
 class NGFragment;
 
 // A class for general block layout (e.g. a <div> with no special style).
 // Lays out the children in sequence.
 class NGBlockLayoutAlgorithm {
  public:
-  NGBlockLayoutAlgorithm(PassRefPtr<const ComputedStyle>, NGBox);
+  NGBlockLayoutAlgorithm(PassRefPtr<const ComputedStyle>, NGBoxIterator);

[cbiesinger, 2016/08/10 19:55:24]

Can you add a comment saying that the iterator is for the children of this
algorithm?

[Gleb Lanbin, 2016/08/10 20:11:44]

Done.

 
   // Actual layout implementation. Lays out the children in sequence within the
   // constraints given by the NGConstraintSpace. Returns a fragment with the
   // resulting layout information.
   // This function can not be const because for interruptible layout, we have
   // to be able to store state information.
   NGFragment* layout(const NGConstraintSpace&);
 
  private:
   RefPtr<const ComputedStyle> m_style;
-  NGBox m_firstChild;
+  NGBoxIterator m_ngBoxIterator;

[cbiesinger, 2016/08/10 19:55:24]

I'd rather us not put the "ng" in the variable name

[eae, 2016/08/10 19:59:32]

agreed. boxIterator will do.

[Gleb Lanbin, 2016/08/10 20:11:44]

Done.

 };
 
 }  // namespace blink
 
 #endif  // NGBlockLayoutAlgorithm_h