JPEG Color profile support was added in bug 48819. Color profile (JPEG_APP0 + 2) markers are currently stored even if color management is disabled. We should only read and retain color profile markers if color management is enabled.
Created attachment 120484 [details]
Comment on attachment 120484 [details]
Clearing flags on attachment: 120484
Committed r103651: <http://trac.webkit.org/changeset/103651>
All reviewed patches have been landed. Closing bug.
fast/images/ycbcr-with-cmyk-color-profile.html started failing after this change. Is this expected?
(In reply to comment #4)
> fast/images/ycbcr-with-cmyk-color-profile.html started failing after this change. Is this expected?
Nope, results look good to me, needs a rebaseline.
Reopening to attach new patch.
Created attachment 120489 [details]
Just noticed http://trac.webkit.org/changeset/103653, need to undo that.
Created attachment 120490 [details]
(In reply to comment #5)
> Is this expected?
Chrome Skia Mac and Chrome CG Mac produce minor paint differences.
Comment on attachment 120490 [details]
Rejecting attachment 120490 [details] from commit-queue.
Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1
ERROR: /mnt/git/webkit-commit-queue/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).
Full output: http://queues.webkit.org/results/11027442
Rebaselined the test in http://trac.webkit.org/changeset/103657.