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

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2006, 2007 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011-2012. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 806 matching lines @@
 }
 
 void LayoutReplaced::computePreferredLogicalWidths() {
   ASSERT(preferredLogicalWidthsDirty());
 
   // We cannot resolve some logical width here (i.e. percent, fill-available or
   // fit-content) as the available logical width may not be set on our
   // containing block.
   const Length& logicalWidth = style()->logicalWidth();
   if (logicalWidth.isPercentOrCalc() || logicalWidth.isFillAvailable() ||
-      logicalWidth.isFitContent())
+      logicalWidth.isFitContent() ||
+      (logicalWidth.isAuto() && (style()->logicalMinWidth().isFillAvailable() ||
+                                 style()->logicalMinWidth().isFitContent() ||

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

Why no isPercentOrCalc() here?

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

umm, not sure, I thought about that. I tried to keep the change as simple as
possible, only addressing the specific issue. However, it mat sense to evaluate
other cases. 

I'll try to figure out other test cases to reproduce the issue so it can get
fixed by this CL as well. 

+                                 style()->logicalMaxWidth().isFillAvailable() ||
+                                 style()->logicalMaxWidth().isFitContent())))
     computeIntrinsicLogicalWidths(m_minPreferredLogicalWidth,
                                   m_maxPreferredLogicalWidth);
   else
     m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth =
         computeReplacedLogicalWidth(ComputePreferred);
 
   const ComputedStyle& styleToUse = styleRef();
   if (styleToUse.logicalWidth().isPercentOrCalc() ||
       styleToUse.logicalMaxWidth().isPercentOrCalc())
     m_minPreferredLogicalWidth = LayoutUnit();
@@ skipping 100 matching lines @@
     inlineBoxWrapper()->root().setHasSelectedChildren(state != SelectionNone);
 }
 
 void LayoutReplaced::IntrinsicSizingInfo::transpose() {
   size = size.transposedSize();
   aspectRatio = aspectRatio.transposedSize();
   std::swap(hasWidth, hasHeight);
 }
 
 }  // namespace blink