Add image color space histograms

Add image color space histograms

Use the same binning as output color spaces, merge them into a helper
function.

This should help inform strategies for image decoding.

BUG=44872
Committed: https://crrev.com/54e9f5e214d9dbc57df09d2669ab2d1a9bb8a780
Cr-Commit-Position: refs/heads/master@{#432636}

[ccameron, 2016-11-15 20:54:04]

ccameron@chromium.org changed reviewers:
+ msarett@chromium.org, pdr@chromium.org, rkaplow@chromium.org

[ccameron, 2016-11-15 20:54:04]

Do histogramming in ImageDecoder::framebufferAtIndex -- this is because not all
decoders specify a color space (and those that can do not always do so), and
ImageDecoder::framebufferAtIndex provides a convenient point that all decoders
hit.

[pdr., 2016-11-15 21:13:57]

LGTM.

Can you do a quick test locally using chrome://histograms/ to double-check that
the expected values come through?

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

msarett@google.com changed reviewers:
+ msarett@google.com

[msarett, 2016-11-15 21:25:27]

LGTM

I'd like to think about histogramming the image/output gamuts as well.  This
will require Skia exposing some more information on SkColorSpace.

[rkaplow, 2016-11-16 06:10:28]

lgtm

[ccameron, 2016-11-16 18:32:46]

The CQ bit was checked by ccameron@chromium.org

[commit-bot: I haz the power, 2016-11-16 18:33:29]

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-11-16 20:22:17]

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

[commit-bot: I haz the power, 2016-11-16 20:22:18]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[ccameron, 2016-11-16 20:26:52]

The CQ bit was checked by ccameron@chromium.org

[commit-bot: I haz the power, 2016-11-16 20:27:32]

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-11-16 21:58:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-16 22:00:45]

Description was changed from

==========
Add image color space histograms

Use the same binning as output color spaces, merge them into a helper
function.

This should help inform strategies for image decoding.

BUG=44872
==========

to

==========
Add image color space histograms

Use the same binning as output color spaces, merge them into a helper
function.

This should help inform strategies for image decoding.

BUG=44872

Committed: https://crrev.com/54e9f5e214d9dbc57df09d2669ab2d1a9bb8a780
Cr-Commit-Position: refs/heads/master@{#432636}
==========

[commit-bot: I haz the power, 2016-11-16 22:00:47]

Patchset 1 (id:??) landed as
https://crrev.com/54e9f5e214d9dbc57df09d2669ab2d1a9bb8a780
Cr-Commit-Position: refs/heads/master@{#432636}