diff options
author | Mike Klein <mtklein@chromium.org> | 2017-11-28 17:08:45 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-11-29 22:43:41 +0000 |
commit | 83c8dd9b1a20ec5b866a6c5a557313c13ae8904f (patch) | |
tree | 10d55288352d41f6fc68313cf617267f2bd83cb5 /src/core | |
parent | 5eaa297267ab2df13598f74425e036c19e9cfaec (diff) |
devirtualize SkCanvas getClipBounds methods
We no longer have any subclasses overriding the virtual hooks,
and we've seen subtle bugs come up back when they did.
In all modes, SkCanvas can answer these queries itself.
BUG=chromium:781238
Change-Id: I37c7511c7bd00c638faacbe4bee89f785691453f
Reviewed-on: https://skia-review.googlesource.com/77202
Reviewed-by: Mike Reed <reed@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/SkCanvas.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp index 3c719013b3..7643c5164b 100644 --- a/src/core/SkCanvas.cpp +++ b/src/core/SkCanvas.cpp @@ -1634,8 +1634,8 @@ bool SkCanvas::quickReject(const SkPath& path) const { return path.isEmpty() || this->quickReject(path.getBounds()); } -SkRect SkCanvas::onGetLocalClipBounds() const { - SkIRect ibounds = this->onGetDeviceClipBounds(); +SkRect SkCanvas::getLocalClipBounds() const { + SkIRect ibounds = this->getDeviceClipBounds(); if (ibounds.isEmpty()) { return SkRect::MakeEmpty(); } @@ -1657,7 +1657,7 @@ SkRect SkCanvas::onGetLocalClipBounds() const { return bounds; } -SkIRect SkCanvas::onGetDeviceClipBounds() const { +SkIRect SkCanvas::getDeviceClipBounds() const { return fMCRec->fRasterClip.getBounds(); } |