Attempting to compile DumpRenderTree on any mac with Xcode 4.4 fails: ../../Tools/DumpRenderTree/chromium/LayoutTestHelper.mm:45:1: error: 'CMProfileLocation' is deprecated [-Werror,-Wdeprecated-declarations] CMProfileLocation initialColorProfileLocation; // The locType field is initialized to 0 which is the same as cmNoProfileBase. ^ ../../Tools/DumpRenderTree/chromium/LayoutTestHelper.mm:56:5: error: 'CMProfileRef' is deprecated [-Werror,-Wdeprecated-declarations] CMProfileRef profile = 0; ^ /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.7.sdk/System/Library/Frameworks/ApplicationServices.framework/Frameworks/ColorSync.framework/Headers/ColorSyncDeprecated.h:1002:41: note: 'CMProfileRef' declared here typedef struct OpaqueCMProfileRef* CMProfileRef DEPRECATED_IN_MAC_OS_X_VERSION_10_6_AND_LATER; We probably shouldn't be using this if it's deprecated in 10.6+
This used to work fine at r123866
While biking home, it occurred to me that this was likely caused by http://crrev.com/149198 . And indeed, after adding mac_deployment_target=10.5 to my GYP_DEFINES, DumpRenderTree builds fine again. So I recommend that as workaround for now. The real fix is to get rid of these deprecated functions.
See bug 80571 for what the apple port does. Apparently the color management functions where deprecated in 10.6 but replacements don't exist until 10.7 :-/ But we could #ifdef on the sdk being 10.7 I suppose, since people building with the 10.7 SDK will have 10.7 and will run locally. Better than not building.
Created attachment 155724 [details] Patch
Note: I haven't tried the attached patch, I just checked that it compiles. It'd be great if someone could check that it produces a DRT that passes pixel tests.
I've confirmed that with this patch I get a DRT and it produces sane looking pixels, but since we don't have cr-mac Mountain Lion baselines I can't tell if they are correct. On many tests the only difference appears to be text rasterization which I'd expect to be different so it looks good.
You can try LayoutTests/fast/scrolling/scrollbar-tickmarks-styled.html . It doesn't have text, but it does have colors, so it would check if the colorspace handling is right.
$ run-webkit-tests --chromium --release fast/scrolling/scrollbar-tickmarks-styled.html wdiff is not installed; please install from MacPorts or elsewhere The test ran as expected.
r? then.
Comment on attachment 155724 [details] Patch Since you copied the code from the mac drt, should you add Apple to the copyright line in the header? View in context: https://bugs.webkit.org/attachment.cgi?id=155724&action=review > Tools/DumpRenderTree/chromium/LayoutTestHelper.mm:44 > + MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_7 this can all go on one line
Created attachment 156360 [details] Patch for landing
Both done, thanks.
Comment on attachment 156360 [details] Patch for landing Clearing flags on attachment: 156360 Committed r124612: <http://trac.webkit.org/changeset/124612>
All reviewed patches have been landed. Closing bug.