RESOLVED INVALID Bug 27460
WebCore/platform/FileSystem.h needs to include wtf/Platform.h before using the PLATFORM macro 
https://bugs.webkit.org/show_bug.cgi?id=27460
Summary WebCore/platform/FileSystem.h needs to include wtf/Platform.h before using th...
Dumitru Daniliuc
Reported 2009-07-20 13:07:14 PDT
FileSystem.h includes wtf/Platform.h after using the PLATFORM macro. Depending on where FileSystem.h is included, PLATFORM might or might not be defined. In general though, wtf/Platform.h should be included at the very top of FileSystem.h.
Attachments
patch (1.01 KB, patch)
2009-07-20 13:20 PDT, Dumitru Daniliuc 		fishd: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dumitru Daniliuc
Comment 1 2009-07-20 13:20:22 PDT
Created attachment 33101 [details] patch
Darin Fisher (:fishd, Google)
Comment 2 2009-07-20 13:29:46 PDT
Comment on attachment 33101 [details] patch This is not correct. The implication of this change is that some .cpp file is not including config.h.
Dumitru Daniliuc
Comment 3 2009-07-20 15:50:33 PDT
The problem is on the Chromium side.
Note You need to log in before you can comment on or make changes to this bug.