switch to SkM44
Change-Id: I2d44eb5b1984212fb461977f5e9ee95717046a8e
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/272467
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
diff --git a/gm/crbug_224618.cpp b/gm/crbug_224618.cpp
index de4f516..dfc759c 100644
--- a/gm/crbug_224618.cpp
+++ b/gm/crbug_224618.cpp
@@ -8,25 +8,12 @@
#include "include/core/SkCanvas.h"
#include "include/core/SkColor.h"
#include "include/core/SkImage.h"
+#include "include/core/SkM44.h"
#include "include/core/SkMatrix.h"
-#include "include/core/SkMatrix44.h"
#include "include/core/SkSurface.h"
#include "include/effects/SkGradientShader.h"
-#include "include/core/SkM44.h"
#include "tools/timer/TimeUtils.h"
-static SkM44 rotate_axis_angle(SkScalar x, SkScalar y, SkScalar z, SkScalar radians) {
- // SkM44 doesn't expose any rotation factories yet
- SkMatrix44 m;
- m.setRotateAboutUnit(x, y, z, radians);
-
- float a[16];
- m.asColMajorf(a);
- SkM44 m4;
- m4.setColMajor(a);
- return m4;
-}
-
// Adapted from https://codepen.io/adamdupuis/pen/qLYzqB
class CrBug224618GM : public skiagm::GM {
public:
@@ -73,7 +60,7 @@
0.f, 0.f, -1.f / radius, 1.f};
SkM44 zoom = SkM44::Translate(0.f, 0.f, radius);
SkM44 postZoom = SkM44::Translate(0.f, 0.f, -radius - 1.f);
- SkM44 rotateHorizontal = rotate_axis_angle(0.f, 1.f, 0.f, 2.356194490192345f);
+ SkM44 rotateHorizontal = SkM44::Rotate({0, 1, 0}, 2.356194490192345f);
// w in degrees will need to be converted to radians
SkV4 axisAngles[6] = {
@@ -94,7 +81,7 @@
};
for (int i = 0; i < 6; ++i) {
- SkM44 model = rotate_axis_angle(axisAngles[i].x, axisAngles[i].y, axisAngles[i].z,
+ SkM44 model = SkM44::Rotate({axisAngles[i].x, axisAngles[i].y, axisAngles[i].z},
SkDegreesToRadians(axisAngles[i].w));
model = SkM44::Translate(radius, radius) * proj * // project and place content
zoom * rotateHorizontal * model * postZoom * // main model matrix