Some source files are compiled unnecessarily in the incremental build. The log of WinCairo port clean build: https://gist.github.com/fujii/1a52f7b02ac4e16927b0b34b4b3eb874#file-webkit-win-cairo-clean-build-txt The log of WinCairo port null incremental build: https://gist.github.com/fujii/1a52f7b02ac4e16927b0b34b4b3eb874#file-webkit-win-cairo-incremental-build-txt In the first clean bulid log, EventTargetInterfaces.h are generated twice in both WebCoreDerivedSources and WebCore projects. > 22>------ Build started: Project: WebCoreDerivedSources, Configuration: Debug x64 ------ > (...) > 22> Generating ../../DerivedSources/WebCore/EventTargetInterfaces.h > (...) > 32>------ Build started: Project: WebCore (WebCore\WebCore), Configuration: Debug x64 ------ > (...) > 32> Generating ../../DerivedSources/WebCore/EventTargetInterfaces.h
Created attachment 321401 [details] Patch
If you want this to be reviewed, please set the r? flag.
Per Arne: Will this change be okay with our internal build? I think so, but you are the expert :-)
(In reply to Brent Fulgham from comment #3) > Per Arne: Will this change be okay with our internal build? I think so, but > you are the expert :-) I think so, since this change is limited to WebCore :)
Comment on attachment 321401 [details] Patch r=me
Comment on attachment 321401 [details] Patch Clearing flags on attachment: 321401 Committed r222395: <http://trac.webkit.org/changeset/222395>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34693106>