diff options
| author |  mtklein <mtklein@chromium.org> | 2015-08-19 17:06:36 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-08-19 17:06:36 -0700 |
| commit | 3679c1bbe7f1e611d7c06b1134e16049799a3ac6 (patch) | |
| tree | c314b690058aa40f3a5e1c7c3303d5da03ba5fdd | |
| parent | 3a30cc1cbb2f42bcbd0695cb994a5f1a52754a89 (diff) | |
Bug fix: we're using SkPMFloat methods on SkColor.
Annoyingly our test bot that forces SkPMFloat_none is a Linux bot using
BGRA SkPMColors, so we'd never notice the bug there.
BUG=skia:
Review URL: https://codereview.chromium.org/1296383006
| -rw-r--r-- | src/opts/SkPMFloat_none.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/opts/SkPMFloat_none.h b/src/opts/SkPMFloat_none.h index 42446e6ac3..adc0bde3b0 100644 --- a/src/opts/SkPMFloat_none.h +++ b/src/opts/SkPMFloat_none.h @@ -37,9 +37,9 @@ inline Sk4f SkPMFloat::alphas() const { inline SkPMFloat SkPMFloat::FromBGRx(SkColor c) { float inv255 = 1.0f / 255; SkPMFloat pmf = SkPMFloat::FromARGB(1.0f, - SkGetPackedR32(c) * inv255, - SkGetPackedG32(c) * inv255, - SkGetPackedB32(c) * inv255); + SkColorGetR(c) * inv255, + SkColorGetG(c) * inv255, + SkColorGetB(c) * inv255); SkASSERT(pmf.isValid()); return pmf; } |