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

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

Issue 2932593004: Update the snap points css properties (Closed)
Patch Set: Fix nits Created 3 years, 6 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
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 383e1e8ee63257b0ffae47d23ca05e74203ce2ff..41e6f5ad2cbb293a9f29961002f7ed9cc3e5d0df 100644
--- a/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
+++ b/third_party/WebKit/Source/core/css/CSSPrimitiveValueMappings.h
@@ -2583,35 +2583,109 @@ inline ScrollBehavior CSSIdentifierValue::ConvertTo() const {
}
template <>
-inline CSSIdentifierValue::CSSIdentifierValue(ScrollSnapType snap_type)
+inline CSSIdentifierValue::CSSIdentifierValue(SnapAxis axis)
: CSSValue(kIdentifierClass) {
- switch (snap_type) {
- case kScrollSnapTypeNone:
- value_id_ = CSSValueNone;
+ switch (axis) {
+ case kSnapAxisX:
+ value_id_ = CSSValueX;
break;
- case kScrollSnapTypeMandatory:
- value_id_ = CSSValueMandatory;
+ case kSnapAxisY:
+ value_id_ = CSSValueY;
+ break;
+ case kSnapAxisBlock:
+ value_id_ = CSSValueBlock;
+ break;
+ case kSnapAxisInline:
+ value_id_ = CSSValueInline;
+ break;
+ case kSnapAxisBoth:
+ value_id_ = CSSValueBoth;
+ break;
+ }
+}
+
+template <>
+inline SnapAxis CSSIdentifierValue::ConvertTo() const {
+ switch (GetValueID()) {
+ case CSSValueX:
+ return kSnapAxisX;
+ case CSSValueY:
+ return kSnapAxisY;
+ case CSSValueBlock:
+ return kSnapAxisBlock;
+ case CSSValueInline:
+ return kSnapAxisInline;
+ case CSSValueBoth:
+ return kSnapAxisBoth;
+ default:
break;
- case kScrollSnapTypeProximity:
+ }
+ NOTREACHED();
+ return kSnapAxisBoth;
+}
+
+template <>
+inline CSSIdentifierValue::CSSIdentifierValue(SnapStrictness strictness)
+ : CSSValue(kIdentifierClass) {
+ switch (strictness) {
+ case kSnapStrictnessProximity:
value_id_ = CSSValueProximity;
break;
+ case kSnapStrictnessMandatory:
+ value_id_ = CSSValueMandatory;
+ break;
}
}
template <>
-inline ScrollSnapType CSSIdentifierValue::ConvertTo() const {
+inline SnapStrictness CSSIdentifierValue::ConvertTo() const {
switch (GetValueID()) {
- case CSSValueNone:
- return kScrollSnapTypeNone;
- case CSSValueMandatory:
- return kScrollSnapTypeMandatory;
case CSSValueProximity:
- return kScrollSnapTypeProximity;
+ return kSnapStrictnessProximity;
+ case CSSValueMandatory:
+ return kSnapStrictnessMandatory;
+ default:
+ break;
+ }
+ NOTREACHED();
+ return kSnapStrictnessProximity;
+}
+
+template <>
+inline CSSIdentifierValue::CSSIdentifierValue(SnapAlignment alignment)
+ : CSSValue(kIdentifierClass) {
+ switch (alignment) {
+ case kSnapAlignmentNone:
+ value_id_ = CSSValueNone;
+ break;
+ case kSnapAlignmentStart:
+ value_id_ = CSSValueStart;
+ break;
+ case kSnapAlignmentEnd:
+ value_id_ = CSSValueEnd;
+ break;
+ case kSnapAlignmentCenter:
+ value_id_ = CSSValueCenter;
+ break;
+ }
+}
+
+template <>
+inline SnapAlignment CSSIdentifierValue::ConvertTo() const {
+ switch (GetValueID()) {
+ case CSSValueNone:
+ return kSnapAlignmentNone;
+ case CSSValueStart:
+ return kSnapAlignmentStart;
+ case CSSValueEnd:
+ return kSnapAlignmentEnd;
+ case CSSValueCenter:
+ return kSnapAlignmentCenter;
default:
break;
}
NOTREACHED();
- return kScrollSnapTypeNone;
+ return kSnapAlignmentNone;
}
template <>

Powered by Google App Engine
This is Rietveld 408576698