Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(192)

Unified Diff: third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h

Issue 2578403002: Changed EVerticalAlign to an enum class and renamed its members (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 0c7b7cddc42ac40716c54786b2f432749bf59aac..5aa992a0c485b355419fb5c24a44a13d947414b1 100644
--- a/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
+++ b/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
@@ -2607,34 +2607,34 @@ template <>
inline CSSIdentifierValue::CSSIdentifierValue(EVerticalAlign a)
: CSSValue(IdentifierClass) {
switch (a) {
- case VerticalAlignTop:
+ case EVerticalAlign::Top:
m_valueID = CSSValueTop;
break;
- case VerticalAlignBottom:
+ case EVerticalAlign::Bottom:
m_valueID = CSSValueBottom;
break;
- case VerticalAlignMiddle:
+ case EVerticalAlign::Middle:
m_valueID = CSSValueMiddle;
break;
- case VerticalAlignBaseline:
+ case EVerticalAlign::Baseline:
m_valueID = CSSValueBaseline;
break;
- case VerticalAlignTextBottom:
+ case EVerticalAlign::TextBottom:
m_valueID = CSSValueTextBottom;
break;
- case VerticalAlignTextTop:
+ case EVerticalAlign::TextTop:
m_valueID = CSSValueTextTop;
break;
- case VerticalAlignSub:
+ case EVerticalAlign::Sub:
m_valueID = CSSValueSub;
break;
- case VerticalAlignSuper:
+ case EVerticalAlign::Super:
m_valueID = CSSValueSuper;
break;
- case VerticalAlignBaselineMiddle:
+ case EVerticalAlign::BaselineMiddle:
m_valueID = CSSValueWebkitBaselineMiddle;
break;
- case VerticalAlignLength:
+ case EVerticalAlign::Length:
m_valueID = CSSValueInvalid;
}
}
@@ -2643,29 +2643,29 @@ template <>
inline EVerticalAlign CSSIdentifierValue::convertTo() const {
switch (m_valueID) {
case CSSValueTop:
- return VerticalAlignTop;
+ return EVerticalAlign::Top;
case CSSValueBottom:
- return VerticalAlignBottom;
+ return EVerticalAlign::Bottom;
case CSSValueMiddle:
- return VerticalAlignMiddle;
+ return EVerticalAlign::Middle;
case CSSValueBaseline:
- return VerticalAlignBaseline;
+ return EVerticalAlign::Baseline;
case CSSValueTextBottom:
- return VerticalAlignTextBottom;
+ return EVerticalAlign::TextBottom;
case CSSValueTextTop:
- return VerticalAlignTextTop;
+ return EVerticalAlign::TextTop;
case CSSValueSub:
- return VerticalAlignSub;
+ return EVerticalAlign::Sub;
case CSSValueSuper:
- return VerticalAlignSuper;
+ return EVerticalAlign::Super;
case CSSValueWebkitBaselineMiddle:
- return VerticalAlignBaselineMiddle;
+ return EVerticalAlign::BaselineMiddle;
default:
break;
}
ASSERT_NOT_REACHED();
- return VerticalAlignTop;
+ return EVerticalAlign::Top;
}
template <>

Powered by Google App Engine
This is Rietveld 408576698