Remove -webkit-column-progression and -webkit-column-axis properties.

Source/core/frame/UseCounter.cpp

 
 /*
  * Copyright (C) 2012 Google, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 246 matching lines @@
     case CSSPropertyWebkitBoxAlign: return 203;
     case CSSPropertyWebkitBoxDirection: return 204;
     case CSSPropertyWebkitBoxFlex: return 205;
     case CSSPropertyWebkitBoxFlexGroup: return 206;
     case CSSPropertyWebkitBoxLines: return 207;
     case CSSPropertyWebkitBoxOrdinalGroup: return 208;
     case CSSPropertyWebkitBoxOrient: return 209;
     case CSSPropertyWebkitBoxPack: return 210;
     case CSSPropertyWebkitBoxReflect: return 211;
     case CSSPropertyWebkitBoxShadow: return 212;
-    case CSSPropertyWebkitColumnAxis: return 214;
+//    case CSSPropertyWebkitColumnAxis: return 214;

[Julien - ping for review, 2014/03/25 18:52:53]

A) We should align the comment with the case statements.
B) We shouldn't have a gap of 4 characters between the start of comment and the
actual text.
C) I much prefer some English comment that explained what was removed + the
value (it's a nit as we are very inconsistent in this file).

[mstensho (USE GERRIT), 2014/03/26 09:03:23]

A&B: Disagrees with how emacs indents by default, but sure, it looked silly. :)
Done.

C: Done. We ought to go through all the removed properties in that file and come
up with some kind of common style.

     case CSSPropertyWebkitColumnBreakAfter: return 215;
     case CSSPropertyWebkitColumnBreakBefore: return 216;
     case CSSPropertyWebkitColumnBreakInside: return 217;
     case CSSPropertyWebkitColumnCount: return 218;
     case CSSPropertyWebkitColumnGap: return 219;
-    case CSSPropertyWebkitColumnProgression: return 220;
+//    case CSSPropertyWebkitColumnProgression: return 220;

[Julien - ping for review, 2014/03/25 18:52:53]

Ditto.

[mstensho (USE GERRIT), 2014/03/26 09:03:23]

Done.

     case CSSPropertyWebkitColumnRule: return 221;
     case CSSPropertyWebkitColumnRuleColor: return 222;
     case CSSPropertyWebkitColumnRuleStyle: return 223;
     case CSSPropertyWebkitColumnRuleWidth: return 224;
     case CSSPropertyWebkitColumnSpan: return 225;
     case CSSPropertyWebkitColumnWidth: return 226;
     case CSSPropertyWebkitColumns: return 227;
 #if defined(ENABLE_CSS_BOX_DECORATION_BREAK) && ENABLE_CSS_BOX_DECORATION_BREAK
     case CSSPropertyWebkitBoxDecorationBreak: return 228;
 #endif
@@ skipping 442 matching lines @@
 UseCounter* UseCounter::getFrom(const StyleSheetContents* sheetContents)
 {
     // FIXME: We may want to handle stylesheets that have multiple owners
     //        http://crbug.com/242125
     if (sheetContents && sheetContents->hasSingleOwnerNode())
         return getFrom(sheetContents->singleOwnerDocument());
     return 0;
 }
 
 } // namespace WebCore