Bug 98778 – [Chromium-Android] Color profiles are not supported on Android

• Component: Images
• Status: RESOLVED WONTFIX

[Peter Beverloo]

Example test:
http://trac.webkit.org/export/130746/trunk/LayoutTests/fast/images/png-with-color-profile.html

Results:
http://build.webkit.org/results/Chromium%20Android%20Release%20(Tests)/r130746%20(1241)/fast/images/png-with-color-profile-diffs.html

The relevant tests have been disabled in TestExpectations.

[Vineet Chaudhary (vineetc)]

Created attachment 182844 [details]
proposed_patch

Color Profiles are now supported on chromium platforms except Android.
This patch enables building code for qcm and iccjpeg for android.

I have tested basic test case attached in the bug which passes in content shell.
Are there any concerns enabling these flags?

[Adam Barth]

Comment on attachment 182844 [details]
proposed_patch

View in context: https://bugs.webkit.org/attachment.cgi?id=182844&action=review

> Source/WTF/wtf/Platform.h:489
> +#define WTF_USE_ICCJPEG 1
> +#define WTF_USE_QCMSLIB 1

Should we just bring these out of the #else clause since they're now enabled on all platforms?

[Vineet Chaudhary (vineetc)]

(In reply to comment #2)
> (From update of attachment 182844 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=182844&action=review
> 
> > Source/WTF/wtf/Platform.h:489
> > +#define WTF_USE_ICCJPEG 1
> > +#define WTF_USE_QCMSLIB 1
> 
> Should we just bring these out of the #else clause since they're now enabled on all platforms?

Yes. I think we can even pull out #define WTF_USE_SKIA 1 along with above, as this is also defined for all ports for Chromium, in same patch or another.

[Vineet Chaudhary (vineetc)]

Created attachment 182867 [details]
updated_patch

I have removed the failing tests from TestExpectations, and checked these tests manually in content shell.

As I have some issues with setup to run /run-webkit-tests --chromium-android

[Adam Barth]

Comment on attachment 182867 [details]
updated_patch

This patch LGTM, but we might want Peter Beverloo to sign off before landing.

[Peter Beverloo]

Hi Vineet, this is awesome, thank you so much! I'll run some performance tests to measure what this means for image decoding time, and will reply with them on the bug once they're complete.

I'm also adding Sami and Alpha as they know much more about image decoding performance on Chromium for Android, and might be able to provide additional insights.

[Vineet Chaudhary (vineetc)]

(In reply to comment #6)
> Hi Vineet, this is awesome, thank you so much! I'll run some performance tests to measure what this means for image decoding time, and will reply with them on the bug once they're complete.
> 
> I'm also adding Sami and Alpha as they know much more about image decoding performance on Chromium for Android, and might be able to provide additional insights.

Hello Peter Beverloo,
Did you got chance to run performance tests? I did dry run for the link
http://regex.info/blog/2012-03-27/1964 found no glitches, but w/o performance data.
If you can point tests which can give perf results I can test that.
Thanks.