Bug 121856 - Make explicit our use of sRGB as the 'untagged' color space
Summary: Make explicit our use of sRGB as the 'untagged' color space
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Horton
URL:
Keywords: InRadar
Depends on: 121857 121859
Blocks: 9567 5972
  Show dependency treegraph
 
Reported: 2013-09-24 13:10 PDT by Tim Horton
Modified: 2017-09-03 01:25 PDT (History)
10 users (show)

See Also:


Attachments
preliminary patch (152.34 KB, patch)
2014-09-11 16:34 PDT, Tim Horton
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 (546.90 KB, application/zip)
2014-09-11 18:09 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (538.27 KB, application/zip)
2014-09-11 19:02 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion (559.03 KB, application/zip)
2014-09-11 19:39 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (616.37 KB, application/zip)
2014-09-11 20:23 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion (616.08 KB, application/zip)
2014-09-11 21:23 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2013-09-24 13:10:36 PDT
On Mountain Lion, CG's default interpretation of DeviceRGB is 'sRGB', whereas CG with 'drawsAsynchronously' on interprets DeviceRGB to be "untagged", flowing unmatched into the context it's painted into. To make everything match up and to make our behavior consistent on all platforms, we should just use sRGB instead of device everywhere, explicitly, matching the CSS and SVG specs.

We need to do some performance testing, and investigate the situation with plugins.
Comment 1 Radar WebKit Bug Importer 2013-09-24 13:11:05 PDT
<rdar://problem/15069231>
Comment 2 Tim Horton 2014-09-11 16:33:10 PDT
On Mavericks, both interpret DeviceRGB as sRGB.
Comment 3 Tim Horton 2014-09-11 16:34:22 PDT
Created attachment 237996 [details]
preliminary patch
Comment 4 WebKit Commit Bot 2014-09-11 16:37:12 PDT
Attachment 237996 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/graphics/cg/ImageBufferCG.cpp:505:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WebCore/platform/graphics/cg/ImageBufferCG.cpp:565:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Total errors found: 2 in 113 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Build Bot 2014-09-11 18:09:40 PDT
Comment on attachment 237996 [details]
preliminary patch

Attachment 237996 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6711950989852672

New failing tests:
svg/css/getComputedStyle-basic.xhtml
accessibility/content-editable-as-textarea.html
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 6 Build Bot 2014-09-11 18:09:43 PDT
Created attachment 238003 [details]
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-16  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 7 Build Bot 2014-09-11 19:02:35 PDT
Comment on attachment 237996 [details]
preliminary patch

Attachment 237996 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5878020812308480

New failing tests:
svg/css/getComputedStyle-basic.xhtml
accessibility/content-editable-as-textarea.html
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 8 Build Bot 2014-09-11 19:02:40 PDT
Created attachment 238005 [details]
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-12  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 9 Build Bot 2014-09-11 19:39:05 PDT
Comment on attachment 237996 [details]
preliminary patch

Attachment 237996 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/4542362218921984

New failing tests:
svg/css/getComputedStyle-basic.xhtml
accessibility/content-editable-as-textarea.html
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 10 Build Bot 2014-09-11 19:39:09 PDT
Created attachment 238009 [details]
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-06  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 11 Build Bot 2014-09-11 20:23:05 PDT
Comment on attachment 237996 [details]
preliminary patch

Attachment 237996 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5785467488305152

New failing tests:
svg/css/getComputedStyle-basic.xhtml
accessibility/content-editable-as-textarea.html
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 12 Build Bot 2014-09-11 20:23:08 PDT
Created attachment 238011 [details]
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-08  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 13 Build Bot 2014-09-11 21:23:01 PDT
Comment on attachment 237996 [details]
preliminary patch

Attachment 237996 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6359109762809856

New failing tests:
svg/css/getComputedStyle-basic.xhtml
accessibility/content-editable-as-textarea.html
fast/css/getComputedStyle/computed-style.html
fast/css/getComputedStyle/computed-style-without-renderer.html
Comment 14 Build Bot 2014-09-11 21:23:05 PDT
Created attachment 238014 [details]
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-04  Port: mac-mountainlion  Platform: Mac OS X 10.8.5