Skip to content

Commit

Permalink
Remove legacy version of SkCanvas::didConcat44 (flutter#17828)
Browse files Browse the repository at this point in the history
Builds flags have been switched, so the old versions aren't needed any
longer.
  • Loading branch information
brianosman committed Apr 21, 2020
1 parent ab52d2d commit 68800a2
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 19 deletions.
4 changes: 0 additions & 4 deletions shell/common/canvas_spy.cc
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,7 @@ void DidDrawCanvas::willRestore() {}

void DidDrawCanvas::didConcat(const SkMatrix& matrix) {}

#ifdef SK_SUPPORT_LEGACY_DIDCONCAT44
void DidDrawCanvas::didConcat44(const SkScalar[]) {}
#else
void DidDrawCanvas::didConcat44(const SkM44&) {}
#endif

void DidDrawCanvas::didScale(SkScalar, SkScalar) {}

Expand Down
4 changes: 0 additions & 4 deletions shell/common/canvas_spy.h
Original file line number Diff line number Diff line change
Expand Up @@ -71,11 +71,7 @@ class DidDrawCanvas final : public SkCanvasVirtualEnforcer<SkNoDrawCanvas> {

// |SkCanvasVirtualEnforcer<SkNoDrawCanvas>|
void didConcat(const SkMatrix&) override;
#ifdef SK_SUPPORT_LEGACY_DIDCONCAT44
void didConcat44(const SkScalar[]) override;
#else
void didConcat44(const SkM44&) override;
#endif
void didScale(SkScalar, SkScalar) override;
void didTranslate(SkScalar, SkScalar) override;

Expand Down
7 changes: 0 additions & 7 deletions testing/mock_canvas.cc
Original file line number Diff line number Diff line change
Expand Up @@ -60,17 +60,10 @@ void MockCanvas::didConcat(const SkMatrix& matrix) {
draw_calls_.emplace_back(DrawCall{current_layer_, ConcatMatrixData{matrix}});
}

#ifdef SK_SUPPORT_LEGACY_DIDCONCAT44
void MockCanvas::didConcat44(const SkScalar matrix[]) {
SkM44 m44 = SkM44::ColMajor(matrix);
draw_calls_.emplace_back(DrawCall{current_layer_, ConcatMatrix44Data{m44}});
}
#else
void MockCanvas::didConcat44(const SkM44& matrix) {
draw_calls_.emplace_back(
DrawCall{current_layer_, ConcatMatrix44Data{matrix}});
}
#endif

void MockCanvas::didScale(SkScalar x, SkScalar y) {
SkMatrix m;
Expand Down
4 changes: 0 additions & 4 deletions testing/mock_canvas.h
Original file line number Diff line number Diff line change
Expand Up @@ -145,11 +145,7 @@ class MockCanvas : public SkCanvasVirtualEnforcer<SkCanvas> {
void willRestore() override;
void didRestore() override {}
void didConcat(const SkMatrix& matrix) override;
#ifdef SK_SUPPORT_LEGACY_DIDCONCAT44
void didConcat44(const SkScalar matrix[]) override;
#else
void didConcat44(const SkM44&) override;
#endif
void didScale(SkScalar x, SkScalar y) override;
void didTranslate(SkScalar x, SkScalar y) override;
void didSetMatrix(const SkMatrix& matrix) override;
Expand Down

0 comments on commit 68800a2

Please sign in to comment.