So that we can make PLATFORM(MAC) false when building for iOS, we need to replace all uses of it that don’t take iOS into account. Most such uses are about building for Cocoa.
Created attachment 223503 [details] Replace PLATFORM(MAC) with equivalent alternatives
Comment on attachment 223503 [details] Replace PLATFORM(MAC) with equivalent alternatives View in context: https://bugs.webkit.org/attachment.cgi?id=223503&action=review > Source/WebCore/platform/graphics/IntRect.h:196 > +#if PLATFORM(MAC) && !PLATFORM(IOS) &&!defined(NSGEOMETRY_TYPES_SAME_AS_CGGEOMETRY_TYPES) &&!defined > Source/WebCore/platform/mac/WebCoreSystemInterface.h:68 > +#if PLATFORM(COCOA) && USE(CA) Is USE(CA) even needed here.
Comment on attachment 223503 [details] Replace PLATFORM(MAC) with equivalent alternatives How did you choose between PLATFORM(COCOA) and USE(APPKIT)?
(In reply to comment #3) > (From update of attachment 223503 [details]) > How did you choose between PLATFORM(COCOA) and USE(APPKIT)? Those aren’t interchangeable: USE(APPKIT) is true only on Mac. I used it in two places: once around a header import, when the header was subsequently used under USE(CG) and USE(APPKIT) conditions; the other was around a forward-declaration of the AppKit class NSImage. That change caused trouble because some WebKit code that was compiling for iOS relied on that class declaration, so I had to fix WebKit in <http://trac.webkit.org/r163660>. Oh, and this change itself was committed in <http://trac.webkit.org/r163657> with another build fix in <http://trac.webkit.org/r163658>.