Remove redundant 'derived constraint space' setters from NGConstraintSpace

third_party/WebKit/Source/core/layout/ng/ng_layout_opportunity_iterator.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_layout_opportunity_iterator.h"
 
 #include "core/layout/ng/ng_physical_constraint_space.h"
 #include "core/layout/ng/ng_units.h"
 #include "wtf/NonCopyingSort.h"
 
@@ skipping 11 matching lines @@
   CollectAllOpportunities(node->bottom, opportunities);
   CollectAllOpportunities(node->right, opportunities);
 }
 
 // Creates layout opportunity from the provided space and the origin point.
 NGLayoutOpportunity CreateLayoutOpportunityFromConstraintSpace(
     const NGConstraintSpace& space,
     const NGLogicalOffset& origin_point) {
   NGLayoutOpportunity opportunity;
   opportunity.offset = space.Offset();
-  opportunity.size = space.Size();
+  opportunity.size = space.AvailableSize();

[cbiesinger, 2016/11/18 23:44:23]

Why is this correct?

[Gleb Lanbin, 2016/11/19 04:20:47]

Acknowledged.

 
   // adjust to the origin_point.
   opportunity.offset += origin_point;
   opportunity.size.inline_size -= origin_point.inline_offset;
   opportunity.size.block_size -= origin_point.block_offset;
   return opportunity;
 }
 
 // Whether 2 edges overlap with each other.
 bool IsOverlapping(const NGEdge& edge1, const NGEdge& edge2) {
@@ skipping 243 matching lines @@
 
 const NGLayoutOpportunity NGLayoutOpportunityIterator::Next() {
   if (opportunity_iter_ == opportunities_.end())
     return NGLayoutOpportunity();
   auto* opportunity = opportunity_iter_;
   opportunity_iter_++;
   return NGLayoutOpportunity(*opportunity);
 }
 
 }  // namespace blink