Changed Order to an enum class and renamed its members

third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h

Index: third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
diff --git a/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h b/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
index 6f80b12fb2b7392860d9d47cd03552cf13daef49..1d634bd9b92df30c231eda7952b8bede9a55480a 100644
--- a/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
+++ b/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
@@ -3546,28 +3546,28 @@ inline CSSIdentifierValue::CSSIdentifierValue(ESpeak e)
 }
 
 template <>
-inline Order CSSIdentifierValue::convertTo() const {
+inline EOrder CSSIdentifierValue::convertTo() const {
   switch (m_valueID) {
     case CSSValueLogical:
-      return LogicalOrder;
+      return EOrder::Logical;
     case CSSValueVisual:
-      return VisualOrder;
+      return EOrder::Visual;
     default:

[napper, 2016/12/05 02:18:22]

I realize this is not part of your CL but this line is redundant

[sashab, 2016/12/05 02:41:18]

The reason it's there I think is otherwise you get a compile error for not
including all the case options in the switch statement :) CSSValueID has like
800 options. These functions are dumb anyway though...

       break;
   }
 
   ASSERT_NOT_REACHED();
-  return LogicalOrder;
+  return EOrder::Logical;
 }
 
 template <>
-inline CSSIdentifierValue::CSSIdentifierValue(Order e)
+inline CSSIdentifierValue::CSSIdentifierValue(EOrder e)
     : CSSValue(IdentifierClass) {
   switch (e) {
-    case LogicalOrder:
+    case EOrder::Logical:
       m_valueID = CSSValueLogical;
       break;
-    case VisualOrder:
+    case EOrder::Visual:
       m_valueID = CSSValueVisual;
       break;
   }