Expose more gamma info for UMA statistics

Expose more gamma info for UMA statistics

TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003

Committed: https://skia.googlesource.com/skia/+/7b9b541af8a0deb0da33a7979f1a904f42db63d1

[msarett, 2016-07-27 16:43:58]

Description was changed from

==========
Alternative method for detecting sRGB gamma tables

BUG=skia:
==========

to

==========
Alternative method for detecting sRGB gamma tables

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

[msarett, 2016-07-27 18:25:33]

Patchset #1 (id:1) has been deleted

[msarett, 2016-07-27 18:25:47]

Description was changed from

==========
Alternative method for detecting sRGB gamma tables

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

to

==========
Hacks to learn more from ColorSpace UMA

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

[msarett, 2016-07-27 18:28:25]

Description was changed from

==========
Hacks to learn more from ColorSpace UMA

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

to

==========
Hacks to learn more from ColorSpace UMA

This should split the non-standard category into 3:
(1) Linear (really means non-standard table)
(2) SRGB (really means non-standard params)
(3) NonStandard (really means non-standard exponent)
    or a situation where R, G, and B don't have
    matching gammas (unlikely).

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

[msarett, 2016-07-27 18:28:55]

msarett@google.com changed reviewers:
+ brianosman@google.com, mtklein@google.com

[msarett, 2016-07-27 19:24:07]

Wrote a new version.  Not sure I like it.

[msarett, 2016-07-27 19:49:24]

Patchset #3 (id:60001) has been deleted

[msarett, 2016-07-27 19:50:19]

Patchset #3 (id:80001) has been deleted

[msarett, 2016-07-27 19:51:02]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-27 19:51:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[msarett, 2016-07-27 19:51:33]

Description was changed from

==========
Hacks to learn more from ColorSpace UMA

This should split the non-standard category into 3:
(1) Linear (really means non-standard table)
(2) SRGB (really means non-standard params)
(3) NonStandard (really means non-standard exponent)
    or a situation where R, G, and B don't have
    matching gammas (unlikely).

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

to

==========
Expose more gamma info for UMA statistics

TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

[msarett, 2016-07-27 19:51:48]

This one is the winner.

[msarett, 2016-07-27 20:02:54]

Store invalid gamma in a separate enum.

[msarett, 2016-07-27 20:03:01]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-27 20:03:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mtklein, 2016-07-27 20:08:35]

lgtm

[commit-bot: I haz the power, 2016-07-27 20:50:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-27 20:50:23]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-07-27 20:50:45]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-07-27 20:50:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-27 20:51:48]

Description was changed from

==========
Expose more gamma info for UMA statistics

TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003
==========

to

==========
Expose more gamma info for UMA statistics

TBR=reed@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2188633003

Committed:
https://skia.googlesource.com/skia/+/7b9b541af8a0deb0da33a7979f1a904f42db63d1
==========

[commit-bot: I haz the power, 2016-07-27 20:51:49]

Committed patchset #4 (id:120001) as
https://skia.googlesource.com/skia/+/7b9b541af8a0deb0da33a7979f1a904f42db63d1