WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
16299
Add a config.h file to DRT
https://bugs.webkit.org/show_bug.cgi?id=16299
Summary
Add a config.h file to DRT
Sam Weinig
Reported
2007-12-04 16:25:36 PST
Now that DumpRenderTree is starting to be used on multiple platforms, it would be a good idea to add a config.h file to ensure platform specific concepts are included everywhere needed.
Attachments
Add a config.h file
(15.43 KB, patch)
2008-10-11 18:21 PDT
,
Alp Toker
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Alp Toker
Comment 1
2008-10-11 18:21:50 PDT
Created
attachment 24295
[details]
Add a config.h file This adds config.h to DumpRenderTree. I've updated the GTK and WIN DRT tools to use the new config.h. Things that still need to be done (and that I don't have the tools to do): * Update the Mac port files * Add config.h to vsproj/xcodeproj files. When adding support for your platform, make sure that the correct config.h is being included in your DRT tool's sources! When config.h has been added to all platforms, this can be removed from DumpRenderTree.h: // FIXME: Remove this when all platforms are using config.h #ifndef Config_H #include <wtf/Platform.h> #endif
Darin Adler
Comment 2
2008-10-14 11:44:08 PDT
Comment on
attachment 24295
[details]
Add a config.h file Landed as
http://trac.webkit.org/changeset/37587
. Clearing review flag.
Darin Adler
Comment 3
2008-10-14 11:44:36 PDT
Sam, do you want to take it from here? Or should we leave this unassigned?
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug