From c4c9f2cb04a7558f78b1339c37b1ca7a0d92570d Mon Sep 17 00:00:00 2001 From: "pkasting@chromium.org" Date: Fri, 10 Oct 2008 01:12:32 +0000 Subject: This change seems dumb, but it works. Apparently, WebCore/html and the DerivedSources both include an HTMLElementFactory.h -- each of which prototype the same functions, but with different signatures >:(. The reason this doesn't seem to be a problem upstream is that their include paths pull in WebCore/html before they pull in the DerivedSources. In our case, these include paths come from two separate .vsprops. The .vsprops seem to be considered last-first, so if we include foo.vsprops and bar.vsprops, the resulting merged include path is "bar/;foo/". So I just swap the .vsprops that contains DerivedSources with the one that contains WebCore/html. There may be a better fix. From the actual list of headers in the upstream projects, it's not clear HTMLElementFactory.h is even still supposed to _be_ autogenerated. If it isn't, and it just hasn't been noticed due to this include path issue, then perhaps we should do a local+upstream patch to stop it from being generated at all. However, I haven't yet been able to figure out why this file gets generated :(. Comments welcome. Review URL: http://codereview.chromium.org/7045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3167 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/build/webkit_common.vsprops | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webkit/build/webkit_common.vsprops b/webkit/build/webkit_common.vsprops index 0fb2e13..15be177 100644 --- a/webkit/build/webkit_common.vsprops +++ b/webkit/build/webkit_common.vsprops @@ -3,6 +3,6 @@ ProjectType="Visual C++" Version="8.00" Name="webkit_common" - InheritedPropertySheets=".\webkit_common_includes.vsprops;.\webkit_common_defines.vsprops;.\js_engine$(JS_ENGINE_TYPE).vsprops;$(SolutionDir)..\third_party\npapi\using_npapi.vsprops;$(SolutionDir)..\skia\using_skia.vsprops;$(SolutionDir)..\build\external_code.vsprops;$(SolutionDir)..\third_party\icu38\build\using_icu.vsprops" + InheritedPropertySheets=".\js_engine$(JS_ENGINE_TYPE).vsprops;.\webkit_common_includes.vsprops;.\webkit_common_defines.vsprops;$(SolutionDir)..\third_party\npapi\using_npapi.vsprops;$(SolutionDir)..\skia\using_skia.vsprops;$(SolutionDir)..\build\external_code.vsprops;$(SolutionDir)..\third_party\icu38\build\using_icu.vsprops" > -- cgit v1.1