If the JavaScripCore folder with forwarding headers contains the inspector folder with derived sources, I get the following compile error: webkitbuild\release\derivedsources\forwardingheaders\javascriptcore\InspectorProtocolObjects.h(304): error C2995: 'WTF::String Inspector::Protocol::InspectorHelpers::getEnumConstantValue(T)': function template has already been defined (compiling source file C:\Projects\WebKit2\OpenSource\Source\WebCore\inspector\InspectorAllInOne.cpp) \webkitbuild\release\derivedsources\forwardingheaders\javascriptcore\inspector\InspectorProtocolObjects.h(301): note: see declaration of 'Inspector::Protocol::InspectorHelpers::getEnumConstantValue' (compiling source file C:\Projects\WebKit2\OpenSource\Source\WebCore\inspector\InspectorAllInOne.cpp) This happens because the file InspectorProtocolObjects.h exists in both folders, and #pragma once will not prevent us from including both files. An include of the type <inspector/InspectorProtocolObjects.h> will include the file from the inspector folder, while <JavaScriptCore/InspectorProtocolObjects.h> will include the file from the JavaScriptCore folder. This can be fixed by putting the WebCore forwarding folder first in the include list. This means that <inspector/InspectorProtocolObjects.h> will also include the file in the JavaScriptCore folder.
Created attachment 289059 [details] Patch
<rdar://problem/28279258>
Windows EWS is seeing this. Does it fix this? C:\cygwin\home\buildbot\WebKit\Source\WebCore\DerivedSources.cpp(345): fatal error C1083: Cannot open include file: 'JSMediaStreamTrackSourcesCallback.cpp': No such file or directory [C:\cygwin\home\buildbot\WebKit\WebKitBuild\Release\Source\WebCore\WebCoreDerivedSources.vcxproj]
That's a separate issue.
I fixed the other issue in r206024
Comment on attachment 289059 [details] Patch Clearing flags on attachment: 289059 Committed r206027: <http://trac.webkit.org/changeset/206027>
All reviewed patches have been landed. Closing bug.
(In reply to comment #5) > I fixed the other issue in r206024 Thanks, Alex!