commit | c891b102867d5af953c6a0ea68f15292bccd5035 | [log] [tgz] |
---|---|---|
author | Brian Osman <brianosman@google.com> | Thu Jun 14 14:50:17 2018 -0400 |
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Fri Jun 15 13:53:05 2018 +0000 |
tree | 68e18b877fdfb3f6576d3dec1dc74dca224c2990 | |
parent | 0e4a12b5259ccb456aaeb14286b793f3a96c842f [diff] |
Simplify usage of GrGLSLColorSpaceXformHelper It was already safe to call setData without (redundantly) checking isValid, so remove those. Made the emitCode helpers safe, too, so that we only need to check for isValid in one spot. Then, because it made more sense, invert the logic and rename it to isNoop(). Pulled out of https://skia-review.googlesource.com/c/skia/+/132090 Bug: skia: Change-Id: I147fa2e0a06c04024c6aa98725fa73501044e4cd Reviewed-on: https://skia-review.googlesource.com/134948 Commit-Queue: Mike Klein <mtklein@google.com> Auto-Submit: Brian Osman <brianosman@google.com> Reviewed-by: Mike Klein <mtklein@google.com>