[webkit-changes] [101241] trunk/Source/WebKit/mac

Mark Rowe mrowe at apple.com
Mon Nov 28 10:26:04 PST 2011


This change appears to be incorrect. WebSystemInterface.mm is a Mac-only file, so it's not clear why you're adding PLATFORM(MAC) to it.

- Mark

On 2011-11-28, at 03:01, hayato at chromium.org wrote:

> Revision
> 101241
> Author
> hayato at chromium.org
> Date
> 2011-11-28 03:01:26 -0800 (Mon, 28 Nov 2011)
> Log Message
> 
> Fix chromium canary build after r101215.
> Unreviewed. Build fix.
> 
> * WebCoreSupport/WebSystemInterface.mm:
> (InitWebCoreSystemInterface):
> Modified Paths
> 
> trunk/Source/WebKit/mac/ChangeLog
> trunk/Source/WebKit/mac/WebCoreSupport/WebSystemInterface.mm
> Diff
> 
> Modified: trunk/Source/WebKit/mac/WebCoreSupport/WebSystemInterface.mm (101240 => 101241)
> 
> --- trunk/Source/WebKit/mac/WebCoreSupport/WebSystemInterface.mm	2011-11-28 10:23:02 UTC (rev 101240)
> +++ trunk/Source/WebKit/mac/WebCoreSupport/WebSystemInterface.mm	2011-11-28 11:01:26 UTC (rev 101241)
> @@ -166,7 +166,7 @@
>      INIT(CreateVMPressureDispatchOnMainQueue);
>  #endif
>  
> -#if !defined(BUILDING_ON_SNOW_LEOPARD) && !defined(BUILDING_ON_LION)
> +#if PLATFORM(MAC) && !defined(BUILDING_ON_SNOW_LEOPARD) && !defined(BUILDING_ON_LION)
>      INIT(GetMacOSXVersionString);
>  #endif
>  
> _______________________________________________
> webkit-changes mailing list
> webkit-changes at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-changes/attachments/20111128/1354d461/attachment.html>


More information about the webkit-changes mailing list