Split EBreak enum into EBreakBetween & EBreakInside

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 1933 matching lines @@
       return EOverflow::kWebkitPagedY;
     default:
       break;
   }
 
   ASSERT_NOT_REACHED();
   return EOverflow::kVisible;
 }
 
 template <>
-inline CSSIdentifierValue::CSSIdentifierValue(EBreak e)
+inline CSSIdentifierValue::CSSIdentifierValue(EBreakBetween e)

[amoylan, 2017/02/07 04:35:02]

Sorted this switch to match enum ordering. Same below.

     : CSSValue(IdentifierClass) {
   switch (e) {
     default:
       ASSERT_NOT_REACHED();
-    case BreakAuto:
+    case EBreakBetween::kAuto:
       m_valueID = CSSValueAuto;
       break;
-    case BreakAvoid:
+    case EBreakBetween::kAvoid:
       m_valueID = CSSValueAvoid;
       break;
-    case BreakAvoidPage:
+    case EBreakBetween::kAvoidColumn:
+      m_valueID = CSSValueAvoidColumn;
+      break;
+    case EBreakBetween::kAvoidPage:
       m_valueID = CSSValueAvoidPage;
       break;
-    case BreakPage:
+    case EBreakBetween::kColumn:
+      m_valueID = CSSValueColumn;
+      break;
+    case EBreakBetween::kLeft:
+      m_valueID = CSSValueLeft;
+      break;
+    case EBreakBetween::kPage:
       m_valueID = CSSValuePage;
       break;
-    case BreakLeft:
-      m_valueID = CSSValueLeft;
+    case EBreakBetween::kRecto:
+      m_valueID = CSSValueRecto;
       break;
-    case BreakRight:
+    case EBreakBetween::kRight:
       m_valueID = CSSValueRight;
       break;
-    case BreakRecto:
-      m_valueID = CSSValueRecto;
-      break;
-    case BreakVerso:
+    case EBreakBetween::kVerso:
       m_valueID = CSSValueVerso;
       break;
-    case BreakAvoidColumn:
-      m_valueID = CSSValueAvoidColumn;
-      break;
-    case BreakColumn:
-      m_valueID = CSSValueColumn;
-      break;
   }
 }
 
 template <>
-inline EBreak CSSIdentifierValue::convertTo() const {
+inline EBreakBetween CSSIdentifierValue::convertTo() const {
   switch (m_valueID) {
     default:
       ASSERT_NOT_REACHED();
     case CSSValueAuto:
-      return BreakAuto;
+      return EBreakBetween::kAuto;
     case CSSValueAvoid:
-      return BreakAvoid;
+      return EBreakBetween::kAvoid;
+    case CSSValueAvoidColumn:
+      return EBreakBetween::kAvoidColumn;
     case CSSValueAvoidPage:
-      return BreakAvoidPage;
+      return EBreakBetween::kAvoidPage;
+    case CSSValueColumn:
+      return EBreakBetween::kColumn;
+    case CSSValueLeft:
+      return EBreakBetween::kLeft;
     case CSSValuePage:
-      return BreakPage;
-    case CSSValueLeft:
-      return BreakLeft;
+      return EBreakBetween::kPage;
+    case CSSValueRecto:
+      return EBreakBetween::kRecto;
     case CSSValueRight:
-      return BreakRight;
-    case CSSValueRecto:
-      return BreakRecto;
+      return EBreakBetween::kRight;
     case CSSValueVerso:
-      return BreakVerso;
-    case CSSValueAvoidColumn:
-      return BreakAvoidColumn;
-    case CSSValueColumn:
-      return BreakColumn;
+      return EBreakBetween::kVerso;
   }
 }
 
+template <>
+inline CSSIdentifierValue::CSSIdentifierValue(EBreakInside e)
+    : CSSValue(IdentifierClass) {
+  switch (e) {
+    default:
+      NOTREACHED();
+    case EBreakInside::kAuto:
+      m_valueID = CSSValueAuto;
+      break;
+    case EBreakInside::kAvoid:
+      m_valueID = CSSValueAvoid;
+      break;
+    case EBreakInside::kAvoidColumn:
+      m_valueID = CSSValueAvoidColumn;
+      break;
+    case EBreakInside::kAvoidPage:
+      m_valueID = CSSValueAvoidPage;
+      break;
+  }
+}
+
+template <>
+inline EBreakInside CSSIdentifierValue::convertTo() const {
+  switch (m_valueID) {
+    default:
+      NOTREACHED();
+    case CSSValueAuto:
+      return EBreakInside::kAuto;
+    case CSSValueAvoid:
+      return EBreakInside::kAvoid;
+    case CSSValueAvoidColumn:
+      return EBreakInside::kAvoidColumn;
+    case CSSValueAvoidPage:
+      return EBreakInside::kAvoidPage;
+  }
+}
+
 template <>
 inline CSSIdentifierValue::CSSIdentifierValue(EPosition e)
     : CSSValue(IdentifierClass) {
   switch (e) {
     case StaticPosition:
       m_valueID = CSSValueStatic;
       break;
     case RelativePosition:
       m_valueID = CSSValueRelative;
       break;
@@ skipping 2755 matching lines @@
     default:
       break;
   }
   ASSERT_NOT_REACHED();
   return ContainsNone;
 }
 
 }  // namespace blink
 
 #endif