Recognize common parametric gamma

Recognize common parametric gamma

5% of images with ICC profiles (in my test set of 10k)
have this exact gamma representation, which is very close
to 2.2f.

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

Committed: https://skia.googlesource.com/skia/+/81e4cc7723ea87de8b8917e52cc25f524225fa43

[msarett, 2016-06-01 20:57:39]

Description was changed from

==========
Recognize common parametric gamma

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

to

==========
Recognize common parametric gamma

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

[msarett, 2016-06-01 21:01:00]

Description was changed from

==========
Recognize common parametric gamma

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

to

==========
Recognize common parametric gamma

5% of images with ICC profiles (in my test set of 10k)
have this exact gamma representation, which is very close
to 2.2f.

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

[msarett, 2016-06-01 21:01:19]

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

[msarett, 2016-06-01 21:01:20]

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp
File src/core/SkColorSpace.cpp (right):

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp#n...
src/core/SkColorSpace.cpp:580: if (color_space_almost_equal(0.9479f, a) &&
Of my set of 10k jpegs with ICC profiles, 500 of them have parametric gammas.

All of the 500 with parametric gammas, have this exact gamma curve.

[scroggo, 2016-06-01 21:23:12]

lgtm

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp
File src/core/SkColorSpace.cpp (right):

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp#n...
src/core/SkColorSpace.cpp:488: uint16_t format = read_big_endian_short(src + 8);
Do these formats have names you can use?

[msarett, 2016-06-01 21:26:13]

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp
File src/core/SkColorSpace.cpp (right):

https://codereview.chromium.org/2027413002/diff/1/src/core/SkColorSpace.cpp#n...
src/core/SkColorSpace.cpp:488: uint16_t format = read_big_endian_short(src + 8);
On 2016/06/01 21:23:12, scroggo wrote:
> Do these formats have names you can use?

They don't, but that'd be nice.  I may name them in a follow-up.

[msarett, 2016-06-01 21:26:32]

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

[commit-bot: I haz the power, 2016-06-01 21:26:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2027413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2027413002/1

[commit-bot: I haz the power, 2016-06-01 21:41:33]

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

[commit-bot: I haz the power, 2016-06-01 21:41:34]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-06-01 21:42:32]

The CQ bit was checked by msarett@google.com

[commit-bot: I haz the power, 2016-06-01 21:42:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2027413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2027413002/1

[commit-bot: I haz the power, 2016-06-01 21:43:39]

Description was changed from

==========
Recognize common parametric gamma

5% of images with ICC profiles (in my test set of 10k)
have this exact gamma representation, which is very close
to 2.2f.

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

to

==========
Recognize common parametric gamma

5% of images with ICC profiles (in my test set of 10k)
have this exact gamma representation, which is very close
to 2.2f.

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

Committed:
https://skia.googlesource.com/skia/+/81e4cc7723ea87de8b8917e52cc25f524225fa43
==========

[commit-bot: I haz the power, 2016-06-01 21:43:39]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/81e4cc7723ea87de8b8917e52cc25f524225fa43