Bug 22561 – Enable WML Support for Windows

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Brent Fulgham]

This update enables support for WML pages in the Windows build of WebKit (both the standard CG and the Cairo variant).  This brings the Windows platform in line with the Mac and Gtk variants.

[Brent Fulgham]

Note that the Windows build already autogenerates the necessary WML support files from the *.in files, since the generic DerivedSources.make and related Perl scripts already do the heavy lifting.

Consequently, this update consists of two changes:
1.  Add the necessary WML files to the build.
2.  Extend the preprocessor macros to include ENABLE_WML.

Once this is done, some testing will be needed to confirm the new format is completely supported.

[Brent Fulgham]

Created attachment 25615 [details]
Add WML files to Windows Build

Note:  For these files to actually execute the ENABLE_WML tag must be added to two places:
1.  The Visual Studio preprocessor definitions in the MSVC project.
2.  The build-generated-files.sh script.

[Adam Roben (:aroben)]

Comment on attachment 25615 [details]
Add WML files to Windows Build

> Index: WebCore/ChangeLog
> ===================================================================
> --- WebCore/ChangeLog	(revision 38852)
> +++ WebCore/ChangeLog	(working copy)
> @@ -1,3 +1,11 @@
> [phone]  Brent Fulgham  <[e-mail]>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Add WML related files to Visual Studio projects.
> +
> +        * WebCore.vcproj/WebCore.vcproj:

You should mention the other changes you made to the vcproj file in the ChangeLog, since the diffs are so hard to read.

> +				PreprocessorDefinitions="__WIN32__;ENABLE_DOM_STORAGE;ENABLE_OFFLINE_WEB_APPLICATIONS;ENABLE_SVG;ENABLE_SVG_ANIMATION;ENABLE_SVG_AS_IMAGE;ENABLE_SVG_USE;ENABLE_SVG_FOREIGN_OBJECT;ENABLE_SVG_FONTS;WEBCORE_CONTEXT_MENUS;ENABLE_VIDEO;ENABLE_WML;ENABLE_WORKERS;ENABLE_XSLT;ENABLE_XPATH"

WML support is disabled by default for all ports right now, so I don't think we should turn it on by default for Windows.

The added files/include paths look fine.

[Brent Fulgham]

Created attachment 25656 [details]
Add WML files to project

Revised patch with better comments about changes.  Please note that the Preprocessor Definition changes are purely from alphabetizing the list.  There are no additions or omissions.

[Adam Roben (:aroben)]

Comment on attachment 25656 [details]
Add WML files to project

r=me

[Brent Fulgham]

Created attachment 25666 [details]
Regenerated VCPROJ against ToT to aid in landing

Reviewed previously.  Diff regenerated to avoid conflicts during landing.

[Matt Lilek]

Committed revision 38897.