Bug 22561

Summary: Enable WML Support for Windows
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Enhancement CC: aroben, ddkilzer
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Add WML files to Windows Build
aroben: review-
Add WML files to project
aroben: review+
Regenerated VCPROJ against ToT to aid in landing none

Description Brent Fulgham 2008-11-29 21:32:10 PST
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.
Comment 1 Brent Fulgham 2008-11-29 21:34:14 PST
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.
Comment 2 Brent Fulgham 2008-11-30 12:24:34 PST
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.
Comment 3 Adam Roben (:aroben) 2008-12-01 13:09:07 PST
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 @@
> +2008-11-30  Brent Fulgham  <bfulgham@gmail.com>
> +
> +        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.
Comment 4 Brent Fulgham 2008-12-01 20:29:15 PST
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.
Comment 5 Adam Roben (:aroben) 2008-12-01 20:31:13 PST
Comment on attachment 25656 [details]
Add WML files to project

r=me
Comment 6 Brent Fulgham 2008-12-01 22:37:54 PST
Created attachment 25666 [details]
Regenerated VCPROJ against ToT to aid in landing

Reviewed previously.  Diff regenerated to avoid conflicts during landing.
Comment 7 Matt Lilek 2008-12-01 22:50:15 PST
Committed revision 38897.