Constrain width/height to minmax values

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

Index: third_party/WebKit/Source/core/layout/ng/ng_length_utils.h
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_length_utils.h b/third_party/WebKit/Source/core/layout/ng/ng_length_utils.h
index 3a1e333f88ccc89dd72a2ca86be8cf46cdb6380d..14028fd4063d08428ce624914a1cce1fa8da40cd 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_length_utils.h
+++ b/third_party/WebKit/Source/core/layout/ng/ng_length_utils.h
@@ -121,6 +121,10 @@ CORE_EXPORT void ApplyAutoMargins(const NGConstraintSpace&,
                                   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