Issue 451 in webp: [webpmux] add icc profile to webp animation success, but did not display color correctly.

[dd448404… via monorail]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 451 by dd448404...@gmail.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451

What steps will reproduce the problem?

set .icc to Animated WebP
1. set DCI-P3-DCI.icc to an animated WebP file
2. success, but did not display color correctly.

set .icc to Solid Webp
1. set DCI-P3-DCI.icc to a WebP file
2. success, display color correctly.


What is the expected output? What do you see instead?
add icc profile to webp animation success, but did not display color correctly.

What version of the product are you using? On what operating system?
1.1.0

Please provide any additional information below.
I just want to know how to make my animated webp file display the correct color.
See their differences: both successfully embedded icc files, but in different colors, the static picture on the right is correct.

Attachments:
截屏2020-01-12下午9.12.04.png 203 KB

--
You received this message because:
1. The project was configured to send all issue notifications to this address

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

[pascal.m… via monorail]

Comment #1 on issue 451 by pascal.m...@gmail.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c1

yes, Color Profile is ignore in libwebp, originally for performance reason (that is: to keep animation light-weight in decoding complexity).
It was foresee that the use-case was too specific to warrant the extra complexity in both code and decoding time.

It is recommended to have the color-profile mapped to sRGB before encoding.

[dd448404… via monorail]

Comment #2 on issue 451 by dd448404...@gmail.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c2

Thank you for your attention to this matter. So color management in WebP only supports static WebP files, not animation WebP files?

In webpmux, I tried to use SET_OPTIONS(-set) to set ICC files for animated WebP files, like this: "-set icc Display_P3.icc in_animation.webp -o icc_container.webp", and successed.

I can extract the correct icc file from the file I exported earlier by using webpmux GET_OPTIONS (-get). So I fully believe WebP has set up the ICC file I want. However, animated WebP file cannot display colors correctly on Chrome, while static pictures can.

Although for performance reasons, I think color management should be allowed on Webp animation, because webpmux provides the option to set color profiles. (But in fact this setting does not work on webp animation.)

Is it possible to display colors correctly?

Thank you again for everything you’ve done.

[jz… via monorail]

Comment #3 on issue 451 by jz...@google.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c3

> yes, Color Profile is ignore in libwebp, originally for performance reason (that is: to keep animation light-weight in decoding complexity).

To be clear, the format supports this, but chrome hasn't implemented support [1].

[1]
https://source.chromium.org/chromium/chromium/src/+/master:third_party/blink/renderer/platform/image-decoders/webp/webp_image_decoder.cc;l=364;drc=7421ea5a0ff0f1e63cdcf8c7c364f6e49701b138?originalUrl=https:%2F%2Fcs.chromium.org%2F
// FIXME: Implement ICC profile support for animated images.

[dd448404… via monorail]

Comment #4 on issue 451 by dd448404...@gmail.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c4

Thanks for your kind reply. Finally, I got your points.😂

[jz… via monorail]

Comment #5 on issue 451 by jz...@google.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c5

I don't think there's an open bug against chrome for this feature, you could file one there (http://crbug.com) to have more people take a look. It might also be worth noting the current behavior of firefox since webp support was recently added there.

[jz… via monorail]

Updates:
Status: Invalid

Comment #6 on issue 451 by jz...@google.com: [webpmux] add icc profile to webp animation success, but did not display color correctly.
https://bugs.chromium.org/p/webp/issues/detail?id=451#c6

I'm going to close this since a bug was filed against Chrome:
https://crbug.com/1042202