Use UMA to track characteristics of dst color spaces

Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces.

BUG=

Committed: https://crrev.com/e61d7832827e6a557a119cc7601821c5fb37bca2
Cr-Commit-Position: refs/heads/master@{#407675}

[msarett, 2016-07-25 15:21:26]

Description was changed from

==========
Use UMA to track characteristics of dst color spaces

BUG=
==========

to

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need to verify
our assumptions about the dst color spaces that we tend
to see. 

BUG=
==========

[msarett, 2016-07-25 15:43:44]

Description was changed from

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need to verify
our assumptions about the dst color spaces that we tend
to see. 

BUG=
==========

to

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces. 

BUG=
==========

[msarett, 2016-07-25 15:45:45]

Patchset #3 (id:40001) has been deleted

[msarett, 2016-07-25 15:49:20]

msarett@google.com changed reviewers:
+ fmalita@chromium.org, holte@chromium.org, reed@chromium.org

[msarett, 2016-07-25 15:49:21]

Please take a look.

fmalita@
third_party/WebKit/Source/platform/

holte@
tools/metrics/histograms/histograms.xml

[f(malita), 2016-07-25 16:13:54]

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.h (right):

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.h:8: #include
"SkColorSpace.h"
third_party/skia/include/core/SkColorSpace.h

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp (right):

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp:320:
sk_sp<SkColorSpace> colorSpace = SkColorSpace::NewICC(profile.data(),
profile.size());
I presume we use Skia because this info is not available via QCMS?

[msarett, 2016-07-25 16:50:52]

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.h (right):

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.h:8: #include
"SkColorSpace.h"
On 2016/07/25 16:13:54, f(malita) wrote:
> third_party/skia/include/core/SkColorSpace.h

Done.

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp (right):

https://codereview.chromium.org/2177933002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp:320:
sk_sp<SkColorSpace> colorSpace = SkColorSpace::NewICC(profile.data(),
profile.size());
On 2016/07/25 16:13:54, f(malita) wrote:
> I presume we use Skia because this info is not available via QCMS?

Yes that's correct!

[f(malita), 2016-07-25 17:12:11]

lgtm

[Steven Holte, 2016-07-25 19:06:30]

lgtm

[msarett, 2016-07-25 19:10:44]

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

[commit-bot: I haz the power, 2016-07-25 19:11:12]

Dry run: 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-25 22:09:38]

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

[commit-bot: I haz the power, 2016-07-25 22:09:39]

Dry run: This issue passed the CQ dry run.

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

The CQ bit was checked by msarett@google.com

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

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-26 01:32:44]

Description was changed from

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces. 

BUG=
==========

to

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces. 

BUG=
==========

[commit-bot: I haz the power, 2016-07-26 01:32:46]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-07-26 01:34:33]

Description was changed from

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces. 

BUG=
==========

to

==========
Use UMA to track characteristics of dst color spaces

As far as I can tell, SRGB gamma and 2.2 gamma dst color
spaces are very common, with all other possibilities
being extremely rare.

We'd like to limit our color correct implementations
to approximations of SRGB and 2Dot2 destinations:
(1) Allows software and gpu to stay in sync.
(2) Avoids code bloat caused by supporting any
    arbitrary representation that can be packed into an
    ICC profile.

To be confident in making this decision, we need some
metrics to help verify our assumptions about the dst color
spaces.

BUG=

Committed: https://crrev.com/e61d7832827e6a557a119cc7601821c5fb37bca2
Cr-Commit-Position: refs/heads/master@{#407675}
==========

[commit-bot: I haz the power, 2016-07-26 01:34:34]

Patchset 4 (id:??) landed as
https://crrev.com/e61d7832827e6a557a119cc7601821c5fb37bca2
Cr-Commit-Position: refs/heads/master@{#407675}