Changed Order to an enum class and renamed its members

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

 /*
  * Copyright (C) 2007 Alexey Proskuryakov <ap@nypop.com>.
  * Copyright (C) 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ skipping 3528 matching lines @@
     case SpeakLiteralPunctuation:
       m_valueID = CSSValueLiteralPunctuation;
       break;
     case SpeakNoPunctuation:
       m_valueID = CSSValueNoPunctuation;
       break;
   }
 }
 
 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;
   }
 }
 
 template <>
 inline ESpeak CSSIdentifierValue::convertTo() const {
   switch (m_valueID) {
     case CSSValueNone:
       return SpeakNone;
@@ skipping 1163 matching lines @@
     default:
       break;
   }
   ASSERT_NOT_REACHED();
   return ContainsNone;
 }
 
 }  // namespace blink
 
 #endif