[LayoutNG] Move createConstraintSpaceFromLayoutObject to NGConstraintSpace

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

Index: third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h
diff --git a/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h b/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h
index d0d9a1fe30c27abcb0203a3f9ee7abe8e8b71698..7d0a2a1058d3aa599d105cda280664f326af0a72 100644
--- a/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h
+++ b/third_party/WebKit/Source/core/layout/ng/NGConstraintSpace.h
@@ -11,10 +11,12 @@
 
 namespace blink {
 
+class NGConstraintSpace;
 class NGDerivedConstraintSpace;
 class NGExclusion;
 class NGFragment;
 class NGLayoutOpportunityIterator;
+class LayoutBox;
 
 enum NGExclusionType {
     NGClearNone = 0,
@@ -50,6 +52,9 @@ public:
         LayoutUnit blockContainerSize);
     ~NGConstraintSpace() { }
 
+    // Constructs Layout NG constraint space from legacy layout object.

[cbiesinger, 2016/08/09 19:26:58]

Can you clarify in the comment that you have to pass the object that needs to be
laid out as the parameter? It may not be immediately clear that it is *not* the
container.

+    static NGConstraintSpace fromLayoutObject(const LayoutBox&);

[eae, 2016/08/09 19:20:22]

Eventually we probably want this (and similar methods) in a legacy_layout_bridge
file or similar but for now this makes sense.

+
     void addExclusion(const NGExclusion, unsigned options = 0);
     void setOverflowTriggersScrollbar(bool inlineTriggers,
         bool blockTriggers);