aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core
diff options
context:
space:
mode:
authorGravatar Mike Klein <mtklein@chromium.org>2017-11-28 17:08:45 -0500
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2017-11-29 22:43:41 +0000
commit83c8dd9b1a20ec5b866a6c5a557313c13ae8904f (patch)
tree10d55288352d41f6fc68313cf617267f2bd83cb5 /src/core
parent5eaa297267ab2df13598f74425e036c19e9cfaec (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.cpp6
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();
}