diff options
author | msarett <msarett@google.com> | 2016-07-27 13:51:46 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-07-27 13:51:46 -0700 |
commit | 7b9b541af8a0deb0da33a7979f1a904f42db63d1 (patch) | |
tree | f9a1f9589dfaba3db1b3e03605dc974040ea1bf5 /src/core/SkColorSpace_ICC.cpp | |
parent | 34ee0c950e1aa24818632084acdb382514f98c08 (diff) |
Expose more gamma info for UMA statistics
TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
Review-Url: https://codereview.chromium.org/2188633003
Diffstat (limited to 'src/core/SkColorSpace_ICC.cpp')
-rw-r--r-- | src/core/SkColorSpace_ICC.cpp | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/src/core/SkColorSpace_ICC.cpp b/src/core/SkColorSpace_ICC.cpp index cbb974f1f1..f8ad47a6cd 100644 --- a/src/core/SkColorSpace_ICC.cpp +++ b/src/core/SkColorSpace_ICC.cpp @@ -560,7 +560,7 @@ static size_t gamma_alloc_size(SkGammas::Type type, const SkGammas::Data& data) static void handle_invalid_gamma(SkGammas::Type* type, SkGammas::Data* data) { if (SkGammas::Type::kNone_Type == *type) { *type = SkGammas::Type::kNamed_Type; - data->fNamed = SkColorSpace::kSRGB_GammaNamed; + data->fNamed = SkColorSpace::kInvalid_GammaNamed; } } @@ -839,6 +839,8 @@ static bool load_a2b0(sk_sp<SkColorLookUpTable>* colorLUT, SkColorSpace::GammaNa (*gammas)->fGreenData = gData; (*gammas)->fBlueData = bData; } + } else { + *gammaNamed = SkColorSpace::kInvalid_GammaNamed; } uint32_t offsetToMatrix = read_big_endian_i32(src + 16); @@ -961,21 +963,21 @@ sk_sp<SkColorSpace> SkColorSpace::NewICC(const void* input, size_t len) { if (tag_equals(r, g, base) && tag_equals(g, b, base)) { SkGammas::Data data; SkGammas::Params params; - SkGammas::Type Type = + SkGammas::Type type = parse_gamma(&data, ¶ms, &tagBytes, r->addr(base), r->fLength); - handle_invalid_gamma(&Type, &data); + handle_invalid_gamma(&type, &data); - if (SkGammas::Type::kNamed_Type == Type) { + if (SkGammas::Type::kNamed_Type == type) { gammaNamed = data.fNamed; } else { - size_t allocSize = sizeof(SkGammas) + gamma_alloc_size(Type, data); + size_t allocSize = sizeof(SkGammas) + gamma_alloc_size(type, data); void* memory = sk_malloc_throw(allocSize); gammas = sk_sp<SkGammas>(new (memory) SkGammas()); - load_gammas(memory, 0, Type, &data, params, r->addr(base)); + load_gammas(memory, 0, type, &data, params, r->addr(base)); - gammas->fRedType = Type; - gammas->fGreenType = Type; - gammas->fBlueType = Type; + gammas->fRedType = type; + gammas->fGreenType = type; + gammas->fBlueType = type; gammas->fRedData = data; gammas->fGreenData = data; @@ -1023,7 +1025,7 @@ sk_sp<SkColorSpace> SkColorSpace::NewICC(const void* input, size_t len) { gammas->fBlueData = bData; } } else { - gammaNamed = kLinear_GammaNamed; + gammaNamed = kInvalid_GammaNamed; } if (kNonStandard_GammaNamed == gammaNamed) { |