diff options
| author |  Mike Reed <reed@google.com> | 2017-08-28 13:32:37 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-08-28 17:54:06 +0000 |
| commit | 3c2d09f89ae119de506722f550a6e28305d4813f (patch) | |
| tree | 15ec5a80e6d3292bef61fea74d8fedf4b8790309 /tools/debugger | |
| parent | de67a2c0e01a68ca8bb3a569947f8e33350f31f7 (diff) | |
change SkRect::growToInclude to take a point instead of x,y
This avoids the dangerous overload problem of
growToInclude(0, 0)
matching to (const SkPoint[], count) rather than growToInclude(x, y)
Bug: skia:
Change-Id: Iaba8b1a579638ff363fde62e4e3004052dd2b2ac
Reviewed-on: https://skia-review.googlesource.com/39501
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Reed <reed@google.com>
Diffstat (limited to 'tools/debugger')
| -rw-r--r-- | tools/debugger/SkDrawCommand.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/debugger/SkDrawCommand.cpp b/tools/debugger/SkDrawCommand.cpp index 8cee07f783..3e05da0572 100644 --- a/tools/debugger/SkDrawCommand.cpp +++ b/tools/debugger/SkDrawCommand.cpp @@ -2741,7 +2741,7 @@ bool SkDrawPointsCommand::render(SkCanvas* canvas) const { bounds.setEmpty(); for (unsigned int i = 0; i < fCount; ++i) { - bounds.growToInclude(fPts[i].fX, fPts[i].fY); + bounds.growToInclude(fPts[i]); } xlate_and_scale_to_bounds(canvas, bounds); |