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

Issue 315743003: Use Identity matrix for SkTransparentShader. (Closed)

Created:
6 years, 6 months ago by scroggo
Modified:
6 years, 6 months ago
Reviewers:
reed1
CC:
skia-review_googlegroups.com
Base URL:
https://skia.googlesource.com/skia.git@master
Visibility:
Public.

Description

Use Identity matrix for SkTransparentShader. The SkTransparentShader is just a placeholder that should not actually be used, so its CTM is not meaningful. However, it needs to be invertible in order to not assert in SkShader::Context's constructor. So use SkMatrix::I(). This fixes a CTS test on Android. Committed: https://skia.googlesource.com/skia/+/ca76c55d60dfe041e30e210d41fa0feb7fb54a1f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M src/core/SkBlitter.cpp View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
scroggo
6 years, 6 months ago (2014-06-03 20:27:34 UTC) #1
reed1
lgtm
6 years, 6 months ago (2014-06-03 22:15:34 UTC) #2
scroggo
The CQ bit was checked by scroggo@google.com
6 years, 6 months ago (2014-06-04 14:05:13 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/scroggo@google.com/315743003/1
6 years, 6 months ago (2014-06-04 14:06:00 UTC) #4
commit-bot: I haz the power
6 years, 6 months ago (2014-06-04 15:21:04 UTC) #5
Message was sent while issue was closed.
Change committed as ca76c55d60dfe041e30e210d41fa0feb7fb54a1f

Powered by Google App Engine
This is Rietveld 408576698