Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(65)

Issue 177073006: add isClipEmpty() -- encourage clients to call this rather than checking clipFoo() results (Closed)

Created:
6 years, 10 months ago by reed1
Modified:
6 years, 10 months ago
Reviewers:
robertphillips
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

add isClipEmpty() -- encourage clients to call this rather than checking clipFoo() results BUG=skia: R=robertphillips@google.com Committed: https://code.google.com/p/skia/source/detail?r=13568

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M include/core/SkCanvas.h View 1 1 chunk +9 lines, -0 lines 0 comments Download
M src/core/SkCanvas.cpp View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
reed1
6 years, 10 months ago (2014-02-24 18:27:07 UTC) #1
robertphillips
lgtm + 1 nit https://codereview.chromium.org/177073006/diff/1/include/core/SkCanvas.h File include/core/SkCanvas.h (right): https://codereview.chromium.org/177073006/diff/1/include/core/SkCanvas.h#newcode979 include/core/SkCanvas.h:979: * result, subsequent calls will ...
6 years, 10 months ago (2014-02-24 19:33:13 UTC) #2
reed1
https://codereview.chromium.org/177073006/diff/1/include/core/SkCanvas.h File include/core/SkCanvas.h (right): https://codereview.chromium.org/177073006/diff/1/include/core/SkCanvas.h#newcode979 include/core/SkCanvas.h:979: * result, subsequent calls will be cheap (until the ...
6 years, 10 months ago (2014-02-24 19:36:18 UTC) #3
reed1
6 years, 10 months ago (2014-02-24 19:38:24 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r13568 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698