Bug 121856

Summary: Make explicit our use of sRGB as the 'untagged' color space
Product: WebKit Reporter: Tim Horton <thorton>
Component: Layout and RenderingAssignee: Tim Horton <thorton>
Status: RESOLVED CONFIGURATION CHANGED    
Severity: Normal CC: bdakin, buildbot, commit-queue, dino, jonlee, mitz, rniwa, sam, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 121857, 121859    
Bug Blocks: 9567, 5972    
Attachments:
Description Flags
preliminary patch
buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2
none
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2
none
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion none

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