From 7b093b77427ab52dc7a43deef3ebbeae378e7b2c Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Mon, 27 Nov 2017 16:30:35 -0500 Subject: remove SkDebugCanvas clip query overrides We don't think these are important, and it's looking like overriding these methods at all is a bad idea. While here we can remove kVizImageHeight and width too, as they're unused. After all these deletions the comment about methods inherited from SkCanvas no longer applies. Change-Id: I036eb9ccca4821516a287daec9847addd529a28c Reviewed-on: https://skia-review.googlesource.com/76480 Reviewed-by: Robert Phillips Commit-Queue: Mike Klein --- tools/debugger/SkDebugCanvas.h | 19 ------------------- 1 file changed, 19 deletions(-) (limited to 'tools/debugger') diff --git a/tools/debugger/SkDebugCanvas.h b/tools/debugger/SkDebugCanvas.h index 72d65d7da7..634d5b416d 100644 --- a/tools/debugger/SkDebugCanvas.h +++ b/tools/debugger/SkDebugCanvas.h @@ -165,25 +165,6 @@ public: Json::Value toJSONOpList(int n, SkCanvas*); - //////////////////////////////////////////////////////////////////////////////// - // Inherited from SkCanvas - //////////////////////////////////////////////////////////////////////////////// - - static const int kVizImageHeight = 256; - static const int kVizImageWidth = 256; - - bool isClipEmpty() const override { return false; } - - bool isClipRect() const override { return true; } - - SkRect onGetLocalClipBounds() const override { - return SkRect::MakeIWH(this->imageInfo().width(), this->imageInfo().height()); - } - - SkIRect onGetDeviceClipBounds() const override { - return SkIRect::MakeWH(this->imageInfo().width(), this->imageInfo().height()); - } - void detachCommands(SkTDArray* dst) { fCommandVector.swap(*dst); } -- cgit v1.2.3