commit 1eb98dba2fc7afdef148145f4b577e8b14deb354
author Florin Malita <fmalita@chromium.org> Fri Jan 26 15:03:38 2018 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Jan 26 20:34:00 2018 +0000
tree 138b56f89457614359a598897d35e63aef9b8bd3
parent c8f740168ff32222a35ab2a3809c7664d7a2432e

[skottie] Add 'rz' rotation support

TBR=

Change-Id: I6dd82e5dc516faac091e5173688d2bc977e4fcaa
Reviewed-on: https://skia-review.googlesource.com/100280
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>

experimental/skottie/Skottie.cpp

diff --git a/experimental/skottie/Skottie.cpp b/experimental/skottie/Skottie.cpp
index 7e56106..efd3d07 100644
--- a/experimental/skottie/Skottie.cpp
+++ b/experimental/skottie/Skottie.cpp
@@ -84,7 +84,14 @@
             [composite](const VectorValue& s) {
                 composite->setScale(ValueTraits<VectorValue>::As<SkVector>(s));
             });
-    auto rotation_attached = BindProperty<ScalarValue>(t["r"], &ctx->fAnimators,
+
+    auto* jrotation = &t["r"];
+    if (jrotation->isNull()) {
+        // 3d rotations have separate rx,ry,rz components.  While we don't fully support them,
+        // we can still make use of rz.
+        jrotation = &t["rz"];
+    }
+    auto rotation_attached = BindProperty<ScalarValue>(*jrotation, &ctx->fAnimators,
             [composite](const ScalarValue& r) {
                 composite->setRotation(r);
             });