Consolidate SkCanvas matrix virtuals.

src/utils/SkLuaCanvas.cpp

Index: src/utils/SkLuaCanvas.cpp
diff --git a/src/utils/SkLuaCanvas.cpp b/src/utils/SkLuaCanvas.cpp
index b5789e058b95cb728882a897b64cf2067a81ec99..676b0e17a40f154723a8c44af50553b665929252 100644
--- a/src/utils/SkLuaCanvas.cpp
+++ b/src/utils/SkLuaCanvas.cpp
@@ -106,35 +106,24 @@ void SkLuaCanvas::willRestore() {
     this->INHERITED::willRestore();
 }
 
-void SkLuaCanvas::didTranslate(SkScalar dx, SkScalar dy) {
-    AUTO_LUA("translate");
-    lua.pushScalar(dx, "dx");
-    lua.pushScalar(dy, "dy");
-    this->INHERITED::didTranslate(dx, dy);
-}
-
-void SkLuaCanvas::didScale(SkScalar sx, SkScalar sy) {
-    AUTO_LUA("scale");
-    lua.pushScalar(sx, "sx");
-    lua.pushScalar(sy, "sy");
-    this->INHERITED::didScale(sx, sy);
-}
-
-void SkLuaCanvas::didRotate(SkScalar degrees) {
-    AUTO_LUA("rotate");
-    lua.pushScalar(degrees, "degrees");
-    this->INHERITED::didRotate(degrees);
-}
-
-void SkLuaCanvas::didSkew(SkScalar kx, SkScalar ky) {
-    AUTO_LUA("skew");
-    lua.pushScalar(kx, "kx");
-    lua.pushScalar(ky, "ky");
-    this->INHERITED::didSkew(kx, ky);
-}
-
 void SkLuaCanvas::didConcat(const SkMatrix& matrix) {
-    AUTO_LUA("concat");
+    switch (matrix.getType()) {

[robertphillips, 2014/03/25 15:44:36]

same here

[f(malita), 2014/03/25 15:58:08]

Done.

+    case SkMatrix::kTranslate_Mask: {
+        AUTO_LUA("translate");
+        lua.pushScalar(matrix.getTranslateX(), "dx");
+        lua.pushScalar(matrix.getTranslateY(), "dy");

[robertphillips, 2014/03/25 15:44:36]

break goes before the } when you do this.

[f(malita), 2014/03/25 15:58:08]

Done.

+    } break;
+    case SkMatrix::kScale_Mask: {
+        AUTO_LUA("scale");
+        lua.pushScalar(matrix.getScaleX(), "sx");
+        lua.pushScalar(matrix.getScaleY(), "sy");
+    } break;
+    default: {
+        AUTO_LUA("concat");
+        lua.pushMatrix(matrix);
+    } break;
+    }
+
     this->INHERITED::didConcat(matrix);
 }