Constrain width/height to minmax values

third_party/WebKit/Source/core/layout/ng/ng_length_utils.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 NGLengthUtils_h
 #define NGLengthUtils_h
 
 #include "core/CoreExport.h"
 #include "core/layout/ng/ng_units.h"
 #include "core/layout/ng/ng_writing_mode.h"
@@ skipping 103 matching lines @@
 
 // Resolves margin: auto in the inline direction after a box has been laid out.
 // This uses the available size from the constraint space and the box size from
 // the fragment to compute the margins that are auto, if any, and adjusts
 // the given NGBoxStrut accordingly.
 CORE_EXPORT void ApplyAutoMargins(const NGConstraintSpace&,
                                   const ComputedStyle&,
                                   const NGFragment&,
                                   NGBoxStrut* margins);
 
+CORE_EXPORT LayoutUnit ConstrainByMinMax(LayoutUnit length,

[cbiesinger, 2017/01/19 22:49:13]

Now that you added this helper, can you make use of it in the other functions in
ng_length_utils.cc as well?

[atotic, 2017/01/19 23:47:44]

done. Found a couple. 

I still dislike lots of template code around ConstrainByMinMax, feels fragile.
Would love to encapsulate it more.

What do you think about creating a LengthResolver object? It'd wrap space and
style, and have bunch of resolution methods: margin, padding, width, height.

+                                         Optional<LayoutUnit> min,
+                                         Optional<LayoutUnit> max);
+
 }  // namespace blink
 
 #endif  // NGLengthUtils_h