[LayoutNG] Initial NGLayoutOpportunityIterator implementation

third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc

 // 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.
 
 #include "core/layout/ng/ng_constraint_space.h"
 
 #include "core/layout/ng/ng_derived_constraint_space.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace blink {
@@ skipping 22 matching lines @@
   EXPECT_TRUE(horz_space->FixedInlineSize());
   EXPECT_TRUE(vert_space->FixedBlockSize());
 
   EXPECT_FALSE(horz_space->FixedBlockSize());
   EXPECT_FALSE(vert_space->FixedInlineSize());
 
   EXPECT_EQ(FragmentColumn, horz_space->BlockFragmentationType());
   EXPECT_EQ(FragmentNone, vert_space->BlockFragmentationType());
 }
 
+TEST(NGConstraintSpaceTest, LayoutOpportunities) {
+  NGPhysicalSize physical_size;
+  physical_size.width = LayoutUnit(600);
+  physical_size.height = LayoutUnit(400);
+  auto* physical_space = new NGPhysicalConstraintSpace(physical_size);
+  auto* space = new NGConstraintSpace(HorizontalTopBottom, physical_space);
+
+  bool for_inline_or_bfc = false;
+  auto iterator = space->LayoutOpportunities(NGClearNone, for_inline_or_bfc);

[cbiesinger, 2016/08/30 15:58:01]

Should this be NGClearFragment...?

+
+  auto firstOppertunity = iterator.Next();

[cbiesinger, 2016/08/30 15:58:01]

Since it's unclear what the type of this variable is, I think you should make it
NGConstraintSpace* instead of auto

Also: opportunity

+  EXPECT_EQ(LayoutUnit(600), firstOppertunity->Size().inline_size);
+  EXPECT_EQ(LayoutUnit(400), firstOppertunity->Size().block_size);
+}
+
 }  // namespace
 
 }  // namespace blink