Bug 27013 – WebCore/platform/FileSystem.h doesn't have a PlatformFileHandle definition for Windows

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Dumitru Daniliuc]

WebCore/platform/FileSystem.h doesn't have a PlatformFileHandle definition for PLATFORM(WIN_OS).

[Dumitru Daniliuc]

What I wanted was a way to distinguish between the CHROMIUM port on Windows and the CHROMIUM port on Mac/Linux. If I understand the code correctly, there's no way to do this in WebKit, because WebKit knows only about one generic CHROMIUM port. So we'll have to take care of this in the Chromium code.

[Dumitru Daniliuc]

After talking to Darin, it looks like we can extend the PlatformFileHandle definition for PLATFORM(WIN) to all WebKit ports built on Windows, which seems reasonable to me and works for Chromium too.

Reopening the issue, patch coming shortly.

[Dumitru Daniliuc]

Created attachment 32399 [details]
patch

[Darin Fisher (:fishd, Google)]

Comment on attachment 32399 [details]
patch

I'm concerned this may cause problems for the PLATFORM(QT) build.
I believe that in the PLATFORM(QT) builds on Windows, PLATFORM(WIN_OS)
will also be defined.

You may be able to avoid that problem by revising the #ifdefs to check
for PLATFORM(QT) before checking for PLATFORM(WIN_OS).

Also, the very last PLATFORM(WIN) in FileSystem.h might be best kept
as PLATFORM(WIN).  It looks like it is a declaration of additional
functions that perhaps only make sense in the PLATFORM(WIN) port
(i.e., Safari on Windows).

[Darin Fisher (:fishd, Google)]

CC'ing folks who seemed to have been involved with the PLATFORM(QT) code in FileSystem.h.  Guys, please see comment #4.  Thanks!

[Dumitru Daniliuc]

Created attachment 32401 [details]
patch

done.

[Darin Fisher (:fishd, Google)]

Comment on attachment 32401 [details]
patch

This looks like it will work.  R=me

[Darin Fisher (:fishd, Google)]

Landed as:  http://trac.webkit.org/changeset/45640