Reimplement min-width: auto

Source/core/css/parser/CSSPropertyParser.cpp

 /*
  * Copyright (C) 2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2012 Adobe Systems Incorporated. All rights reserved.
  * Copyright (C) 2012 Intel Corporation. All rights reserved.
  *
@@ skipping 806 matching lines @@
     case CSSPropertyWebkitMaxLogicalHeight:
         validPrimitive = (id == CSSValueNone || validWidthOrHeight(value, unitless));
         break;
 
     case CSSPropertyMinWidth:
     case CSSPropertyMinHeight:
         unitless = FUnitlessQuirk;
         // fall through
     case CSSPropertyWebkitMinLogicalWidth:
     case CSSPropertyWebkitMinLogicalHeight:
-        validPrimitive = validWidthOrHeight(value, unitless);
+        validPrimitive = (id == CSSValueAuto || validWidthOrHeight(value, unitless));

[Julien - ping for review, 2015/04/06 17:06:47]

Unneeded parentheses.

[cbiesinger, 2015/04/06 21:34:16]

Yes, but this matches the style in the cases around it, for
CSSPropertyWebkitMaxLogicalHeight above and CSSPropertyWebkitLogicalHeight
below.

[Julien - ping for review, 2015/04/13 21:25:59]

This goes against our usual convention. I would argue that it doesn't matter
what the rest of the code in the file looks like, we want consistency across the
codebase not a single file.

[cbiesinger, 2015/04/13 23:23:12]

Hmm ok, removed the parentheses.

         break;
 
     case CSSPropertyWidth:
     case CSSPropertyHeight:
         unitless = FUnitlessQuirk;
         // fall through
     case CSSPropertyWebkitLogicalWidth:
     case CSSPropertyWebkitLogicalHeight:
         validPrimitive = (id == CSSValueAuto || validWidthOrHeight(value, unitless));
         break;
@@ skipping 7634 matching lines @@
         }
     }
 
     if (!list->length())
         return nullptr;
 
     return list.release();
 }
 
 } // namespace blink