[css-sizing] Ensure positive available size on replaced elements

third_party/WebKit/Source/core/layout/LayoutBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 3232 matching lines @@
   return computeReplacedLogicalWidthRespectingMinMaxWidth(
       computeReplacedLogicalWidthUsing(MainOrPreferredSize,
                                        style()->logicalWidth()),
       shouldComputePreferred);
 }
 
 LayoutUnit LayoutBox::computeReplacedLogicalWidthRespectingMinMaxWidth(
     LayoutUnit logicalWidth,
     ShouldComputePreferred shouldComputePreferred) const {
   LayoutUnit minLogicalWidth = (shouldComputePreferred == ComputePreferred &&
-                                style()->logicalMinWidth().isPercentOrCalc())
+                                (style()->logicalMinWidth().isPercentOrCalc() ||
+                                 style()->logicalMinWidth().isFitContent() ||
+                                 style()->logicalMinWidth().isFillAvailable()))
                                    ? logicalWidth
                                    : computeReplacedLogicalWidthUsing(
                                          MinSize, style()->logicalMinWidth());
   LayoutUnit maxLogicalWidth =
       (shouldComputePreferred == ComputePreferred &&
-       style()->logicalMaxWidth().isPercentOrCalc()) ||
+       (style()->logicalMaxWidth().isPercentOrCalc() ||
+        style()->logicalMaxWidth().isFitContent() ||
+        style()->logicalMaxWidth().isFillAvailable())) ||

[mstensho (USE GERRIT), 2016/11/10 19:54:25]

So, what's a good term to use when you're either percent, calc, fit-content or
fill-available? :)

It would be nice to introduce a Length::isSomethingBlah() for this, but maybe we
don't need to figure it out as part of this CL.

[jfernandez, 2016/11/10 20:54:39]

If I manage to do something very simple, I think it'd make sense to do it as
part of
this CL, so the chance becomes simpler and clearer. 

               style()->logicalMaxWidth().isMaxSizeNone()
           ? logicalWidth
           : computeReplacedLogicalWidthUsing(MaxSize,
                                              style()->logicalMaxWidth());
   return std::max(minLogicalWidth, std::min(logicalWidth, maxLogicalWidth));
 }
 
 LayoutUnit LayoutBox::computeReplacedLogicalWidthUsing(
     SizeType sizeType,
     const Length& logicalWidth) const {
@@ skipping 2368 matching lines @@
   LayoutRect rect = frameRect();
 
   LayoutBlock* block = containingBlock();
   if (block)
     block->adjustChildDebugRect(rect);
 
   return rect;
 }
 
 }  // namespace blink