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

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

Issue 2380233002: Changed EListStylePosition to an enum class and renamed its members to keywords (Closed)
Patch Set: Rebase Created 4 years, 2 months 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutListMarker.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 d9d73070613a0457d0fc82ca8452d79ee5ac9beb..a254eed079aac705a5307fb046e35e0f98384306 100644
--- a/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
+++ b/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
@@ -1425,10 +1425,10 @@ template <>
inline CSSIdentifierValue::CSSIdentifierValue(EListStylePosition e)
: CSSValue(IdentifierClass) {
switch (e) {
- case ListStylePositionOutside:
+ case EListStylePosition::Outside:
m_valueID = CSSValueOutside;
break;
- case ListStylePositionInside:
+ case EListStylePosition::Inside:
m_valueID = CSSValueInside;
break;
}
@@ -1438,15 +1438,15 @@ template <>
inline EListStylePosition CSSIdentifierValue::convertTo() const {
switch (m_valueID) {
case CSSValueOutside:
- return ListStylePositionOutside;
+ return EListStylePosition::Outside;
case CSSValueInside:
- return ListStylePositionInside;
+ return EListStylePosition::Inside;
default:
break;
}
ASSERT_NOT_REACHED();
- return ListStylePositionOutside;
+ return EListStylePosition::Outside;
}
template <>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutListMarker.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698