[webkit-reviews] review granted: [Bug 135807] generate header detection files on Windows CMake : [Attachment 236383] Patch

[bugzilla-daemon at webkit.org]

Brent Fulgham <bfulgham at webkit.org> has granted Alex Christensen
<achristensen at apple.com>'s request for review:
Bug 135807: generate header detection files on Windows CMake
https://bugs.webkit.org/show_bug.cgi?id=135807

Attachment 236383: Patch
https://bugs.webkit.org/attachment.cgi?id=236383&action=review

------- Additional Comments from Brent Fulgham <bfulgham at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=236383&action=review


r=me. Please change those bugs to be "FIXME(Bug ###)" format, with bugs filed
to extend the CMake stuff to do what we need for internal use.

> Source/WTF/wtf/PlatformWin.cmake:22
> +#FIXME: This should run testOSXLevel.cmd if it is available.

Please file a bug and make this a #FIXME(Bug##) bug.

> Source/WebCore/PlatformWin.cmake:93
> +#FIXME: This should test if AVF headers are available.

Ditto above comment.