Parse "-webkit-columns: auto <length>" properly.

Source/core/css/StylePropertyShorthand.h

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2008 Apple Inc. 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.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 14 matching lines @@
 
 namespace WebCore {
 
 class StylePropertyShorthand {
     WTF_MAKE_FAST_ALLOCATED;
 public:
     StylePropertyShorthand()
         : m_properties(0)
         , m_propertiesForInitialization(0)
         , m_length(0)
+        , m_allAcceptAuto(false)
     {
     }
 
-    StylePropertyShorthand(const CSSPropertyID* properties, unsigned numProperties)
+    StylePropertyShorthand(const CSSPropertyID* properties, unsigned numProperties, bool allAcceptAuto = false)

[Julien - ping for review, 2013/07/23 00:56:29]

NO!

This class represents a generic shorthand, not a dumping ground for random extra
field to handle special cases.

Also, booleans are usually frowned upon as they make the code less readable.

[mstensho (USE GERRIT), 2013/07/23 14:19:07]

Done.

Also, my patch doesn't even apply anymore, since StylePropertyShorthand.h is
gone and is now auto-generated. :)

         : m_properties(properties)
         , m_propertiesForInitialization(0)
         , m_length(numProperties)
+        , m_allAcceptAuto(allAcceptAuto)
     {
     }
 
-    StylePropertyShorthand(const CSSPropertyID* properties, const StylePropertyShorthand** propertiesForInitialization, unsigned numProperties)
+    StylePropertyShorthand(const CSSPropertyID* properties, const StylePropertyShorthand** propertiesForInitialization, unsigned numProperties, bool allAcceptAuto = false)
         : m_properties(properties)
         , m_propertiesForInitialization(propertiesForInitialization)
         , m_length(numProperties)
+        , m_allAcceptAuto(allAcceptAuto)
     {
     }
 
     const CSSPropertyID* properties() const { return m_properties; }
     const StylePropertyShorthand** propertiesForInitialization() const { return m_propertiesForInitialization; }
     unsigned length() const { return m_length; }
+    bool allAcceptAuto() const { return m_allAcceptAuto; }
 
 private:
     const CSSPropertyID* m_properties;
     const StylePropertyShorthand** m_propertiesForInitialization;
     unsigned m_length;
+    bool m_allAcceptAuto;
 };
 
 const StylePropertyShorthand& backgroundShorthand();
 const StylePropertyShorthand& backgroundPositionShorthand();
 const StylePropertyShorthand& backgroundRepeatShorthand();
 const StylePropertyShorthand& borderShorthand();
 const StylePropertyShorthand& borderAbridgedShorthand();
 const StylePropertyShorthand& borderBottomShorthand();
 const StylePropertyShorthand& borderColorShorthand();
 const StylePropertyShorthand& borderImageShorthand();
@@ skipping 37 matching lines @@
 #endif
 
 // Returns an empty list if the property is not a shorthand
 const StylePropertyShorthand& shorthandForProperty(CSSPropertyID);
 
 bool isExpandedShorthand(CSSPropertyID);
 
 } // namespace WebCore
 
 #endif // StylePropertyShorthand_h