Revert "Implement new SkCanvas virtuals in testing canvases (flutter#15893)"

This reverts commit 92ad31a.

Author: NoamDev

shell/common/canvas_spy.cc

@@ -53,12 +53,6 @@ void DidDrawCanvas::willRestore() {}
 
 void DidDrawCanvas::didConcat(const SkMatrix& matrix) {}
 
-void DidDrawCanvas::didConcat44(const SkScalar[]) {}
-
-void DidDrawCanvas::didScale(SkScalar, SkScalar) {}
-
-void DidDrawCanvas::didTranslate(SkScalar, SkScalar) {}
-
 void DidDrawCanvas::didSetMatrix(const SkMatrix& matrix) {}
 
 void DidDrawCanvas::onClipRect(const SkRect& rect,

shell/common/canvas_spy.h

@@ -71,9 +71,6 @@ class DidDrawCanvas final : public SkCanvasVirtualEnforcer<SkNoDrawCanvas> {
 
   // |SkCanvasVirtualEnforcer<SkNoDrawCanvas>|
   void didConcat(const SkMatrix&) override;
-  void didConcat44(const SkScalar[]) override;
-  void didScale(SkScalar, SkScalar) override;
-  void didTranslate(SkScalar, SkScalar) override;
 
   // |SkCanvasVirtualEnforcer<SkNoDrawCanvas>|
   void didSetMatrix(const SkMatrix&) override;

testing/mock_canvas.cc

@@ -60,24 +60,6 @@ void MockCanvas::didConcat(const SkMatrix& matrix) {
   draw_calls_.emplace_back(DrawCall{current_layer_, ConcatMatrixData{matrix}});
 }
 
-void MockCanvas::didConcat44(const SkScalar matrix[]) {
-  SkMatrix44 m44;
-  m44.setColMajor(matrix);
-  draw_calls_.emplace_back(DrawCall{current_layer_, ConcatMatrix44Data{m44}});
-}
-
-void MockCanvas::didScale(SkScalar x, SkScalar y) {
-  SkMatrix m;
-  m.setScale(x, y);
-  this->didConcat(m);
-}
-
-void MockCanvas::didTranslate(SkScalar x, SkScalar y) {
-  SkMatrix m;
-  m.setTranslate(x, y);
-  this->didConcat(m);
-}
-
 void MockCanvas::didSetMatrix(const SkMatrix& matrix) {
   draw_calls_.emplace_back(DrawCall{current_layer_, SetMatrixData{matrix}});
 }
@@ -364,16 +346,6 @@ std::ostream& operator<<(std::ostream& os,
   return os << data.matrix;
 }
 
-bool operator==(const MockCanvas::ConcatMatrix44Data& a,
-                const MockCanvas::ConcatMatrix44Data& b) {
-  return a.matrix == b.matrix;
-}
-
-std::ostream& operator<<(std::ostream& os,
-                         const MockCanvas::ConcatMatrix44Data& data) {
-  return os << data.matrix;
-}
-
 bool operator==(const MockCanvas::SetMatrixData& a,
                 const MockCanvas::SetMatrixData& b) {
   return a.matrix == b.matrix;

testing/mock_canvas.h

@@ -16,7 +16,6 @@
 #include "third_party/skia/include/core/SkClipOp.h"
 #include "third_party/skia/include/core/SkData.h"
 #include "third_party/skia/include/core/SkImageFilter.h"
-#include "third_party/skia/include/core/SkMatrix44.h"
 #include "third_party/skia/include/core/SkPath.h"
 #include "third_party/skia/include/core/SkRRect.h"
 #include "third_party/skia/include/core/SkRect.h"
@@ -56,10 +55,6 @@ class MockCanvas : public SkCanvasVirtualEnforcer<SkCanvas> {
     SkMatrix matrix;
   };
 
-  struct ConcatMatrix44Data {
-    SkMatrix44 matrix;
-  };
-
   struct SetMatrixData {
     SkMatrix matrix;
   };
@@ -114,7 +109,6 @@ class MockCanvas : public SkCanvasVirtualEnforcer<SkCanvas> {
                                     SaveLayerData,
                                     RestoreData,
                                     ConcatMatrixData,
-                                    ConcatMatrix44Data,
                                     SetMatrixData,
                                     DrawRectData,
                                     DrawPathData,
@@ -145,9 +139,6 @@ class MockCanvas : public SkCanvasVirtualEnforcer<SkCanvas> {
   void willRestore() override;
   void didRestore() override {}
   void didConcat(const SkMatrix& matrix) override;
-  void didConcat44(const SkScalar matrix[]) override;
-  void didScale(SkScalar x, SkScalar y) override;
-  void didTranslate(SkScalar x, SkScalar y) override;
   void didSetMatrix(const SkMatrix& matrix) override;
 
   // Draw and clip operations that we track.
@@ -278,10 +269,6 @@ extern bool operator==(const MockCanvas::ConcatMatrixData& a,
                        const MockCanvas::ConcatMatrixData& b);
 extern std::ostream& operator<<(std::ostream& os,
                                 const MockCanvas::ConcatMatrixData& data);
-extern bool operator==(const MockCanvas::ConcatMatrix44Data& a,
-                       const MockCanvas::ConcatMatrix44Data& b);
-extern std::ostream& operator<<(std::ostream& os,
-                                const MockCanvas::ConcatMatrix44Data& data);
 extern bool operator==(const MockCanvas::SetMatrixData& a,
                        const MockCanvas::SetMatrixData& b);
 extern std::ostream& operator<<(std::ostream& os,