Changed TextDirection to an enum class and renamed its members

third_party/WebKit/Source/core/html/forms/RangeInputType.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Apple 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 172 matching lines @@
 
   // FIXME: We can't use stepUp() for the step value "any". So, we increase
   // or decrease the value by 1/100 of the value range. Is it reasonable?
   const Decimal step =
       equalIgnoringCase(element().fastGetAttribute(stepAttr), "any")
           ? (stepRange.maximum() - stepRange.minimum()) / 100
           : stepRange.step();
   const Decimal bigStep =
       std::max((stepRange.maximum() - stepRange.minimum()) / 10, step);
 
-  TextDirection dir = LTR;
+  TextDirection dir = TextDirection::Ltr;
   bool isVertical = false;
   if (element().layoutObject()) {
     dir = computedTextDirection();
     ControlPart part = element().layoutObject()->style()->appearance();
     isVertical = part == SliderVerticalPart;
   }
 
   Decimal newValue;
-  if (key == "ArrowUp")
+  if (key == "ArrowUp") {
     newValue = current + step;
-  else if (key == "ArrowDown")
+  } else if (key == "ArrowDown") {
     newValue = current - step;
-  else if (key == "ArrowLeft")
-    newValue = (isVertical || dir == RTL) ? current + step : current - step;
-  else if (key == "ArrowRight")
-    newValue = (isVertical || dir == RTL) ? current - step : current + step;
-  else if (key == "PageUp")
+  } else if (key == "ArrowLeft") {
+    newValue = (isVertical || dir == TextDirection::Rtl) ? current + step
+                                                         : current - step;
+  } else if (key == "ArrowRight") {
+    newValue = (isVertical || dir == TextDirection::Rtl) ? current - step
+                                                         : current + step;
+  } else if (key == "PageUp") {
     newValue = current + bigStep;
-  else if (key == "PageDown")
+  } else if (key == "PageDown") {
     newValue = current - bigStep;
-  else if (key == "Home")
+  } else if (key == "Home") {
     newValue = isVertical ? stepRange.maximum() : stepRange.minimum();
-  else if (key == "End")
+  } else if (key == "End") {
     newValue = isVertical ? stepRange.minimum() : stepRange.maximum();
-  else
+  } else {
     return;  // Did not match any key binding.
+  }
 
   newValue = stepRange.clampValue(newValue);
 
   if (newValue != current) {
     EventQueueScope scope;
     TextFieldEventBehavior eventBehavior = DispatchInputAndChangeEvent;
     setValueAsDecimal(newValue, eventBehavior, IGNORE_EXCEPTION);
 
     if (AXObjectCache* cache = element().document().existingAXObjectCache())
       cache->handleValueChanged(&element());
@@ skipping 171 matching lines @@
                                      : Decimal::infinity(Decimal::Negative);
   const Decimal closestRight = middle != m_tickMarkValues.size()
                                    ? m_tickMarkValues[middle]
                                    : Decimal::infinity(Decimal::Positive);
   if (closestRight - value < value - closestLeft)
     return closestRight;
   return closestLeft;
 }
 
 }  // namespace blink