[layoutng] Support computing min-content and max-content

third_party/WebKit/Source/core/layout/ng/ng_physical_fragment_base.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 NGPhysicalFragmentBase_h
 #define NGPhysicalFragmentBase_h
 
 #include "core/CoreExport.h"
 #include "core/layout/ng/ng_constraint_space.h"
 #include "core/layout/ng/ng_units.h"
@@ skipping 20 matching lines @@
   // The accessors in this class shouldn't be used by layout code directly,
   // instead should be accessed by the NGFragmentBase classes. These accessors
   // exist for paint, hit-testing, etc.
 
   // Returns the border-box size.
   NGPhysicalSize Size() const { return size_; }
   LayoutUnit Width() const { return size_.width; }
   LayoutUnit Height() const { return size_.height; }
 
   // Returns the total size, including the contents outside of the border-box.
+  NGPhysicalSize OverflowSize() { return overflow_; }

[Gleb Lanbin, 2016/10/31 19:52:47]

shouldn't a getter function name for variable have the same name as the
variable?

[cbiesinger, 2016/11/01 19:10:04]

It turns out I no longer need this function, so I removed it.

   LayoutUnit WidthOverflow() const { return overflow_.width; }
   LayoutUnit HeightOverflow() const { return overflow_.height; }
 
   // Returns the offset relative to the parent fragement's content-box.
   LayoutUnit LeftOffset() const {
     DCHECK(has_been_placed_);
     return offset_.left;
   }
 
   LayoutUnit TopOffset() const {
@@ skipping 24 matching lines @@
   NGPhysicalSize overflow_;
   NGPhysicalOffset offset_;
 
   unsigned type_ : 1;
   unsigned has_been_placed_ : 1;
 };
 
 }  // namespace blink
 
 #endif  // NGFragmentBase_h