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

Unified Diff: third_party/WebKit/LayoutTests/typedcssom/inlinestyle/properties/transform.html

Issue 2546343002: [CSS Typed OM] Add additional test capability for string->CSSStyleValue in property-suite (Closed)
Patch Set: fix location of quotes 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/LayoutTests/typedcssom/inlinestyle/properties/transform.html
diff --git a/third_party/WebKit/LayoutTests/typedcssom/inlinestyle/properties/transform.html b/third_party/WebKit/LayoutTests/typedcssom/inlinestyle/properties/transform.html
index e084c820e93c67b3a0f957752d086f842671c0e6..5b8b1be019f65b19f63b0fe8392c1fe43d523c4d 100644
--- a/third_party/WebKit/LayoutTests/typedcssom/inlinestyle/properties/transform.html
+++ b/third_party/WebKit/LayoutTests/typedcssom/inlinestyle/properties/transform.html
@@ -41,10 +41,45 @@ runInlineStylePropertyMapTests( {
// Perspectives
// TODO(meade)
],
+ // Values with these strings aren't used in Typed OM, but can also be
+ // represented by the specified values.
+ validStringMappings: {
+ // Rotations
+ 'rotateX(45deg)':
+ cssTransformWithRotate3D(1, 0, 0, new CSSAngleValue(45, 'deg')),
+ 'rotateX(1rad)':
+ cssTransformWithRotate3D(1, 0, 0, new CSSAngleValue(1, 'rad')),
+ 'rotateX(6.2grad)':
+ cssTransformWithRotate3D(1, 0, 0, new CSSAngleValue(6.2, 'grad')),
+ 'rotateX(0.31turn)':
+ cssTransformWithRotate3D(1, 0, 0, new CSSAngleValue(0.31, 'turn')),
+ 'rotateY(45deg)':
+ cssTransformWithRotate3D(0, 1, 0, new CSSAngleValue(45, 'deg')),
+ 'rotateY(1rad)':
+ cssTransformWithRotate3D(0, 1, 0, new CSSAngleValue(1, 'rad')),
+ 'rotateY(6.2grad)':
+ cssTransformWithRotate3D(0, 1, 0, new CSSAngleValue(6.2, 'grad')),
+ 'rotateY(0.31turn)':
+ cssTransformWithRotate3D(0, 1, 0, new CSSAngleValue(0.31, 'turn')),
+ 'rotateZ(45deg)':
+ cssTransformWithRotate3D(0, 0, 1, new CSSAngleValue(45, 'deg')),
+ 'rotateZ(1rad)':
+ cssTransformWithRotate3D(0, 0, 1, new CSSAngleValue(1, 'rad')),
+ 'rotateZ(6.2grad)':
+ cssTransformWithRotate3D(0, 0, 1, new CSSAngleValue(6.2, 'grad')),
+ 'rotateZ(0.31turn)':
+ cssTransformWithRotate3D(0, 0, 1, new CSSAngleValue(0.31, 'turn')),
+ },
supportsMultiple: false,
invalidObjects: [new CSSSimpleLength(4, 'px')]
});
+// TODO(meade): Remove/update this test once translate is supported.
+test(function() {
+ testElement.style.transform = 'translateY(50px)';
+ testElement.styleMap.entries();
+}, "Unhandled case doesn't crash.");
+
test(function() {
testElement.style.transform = 'rotate(calc(45deg + 1rad))';
let result = testElement.styleMap.get('transform');

Powered by Google App Engine
This is Rietveld 408576698