Make more use of USE(OPENGL_ES) where it makes sense
Created attachment 334247 [details] Patch
<rdar://problem/37699443>
Comment on attachment 334247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=334247&action=review > Source/WebCore/platform/graphics/GraphicsContext3D.h:64 > +#else // PLATFORM(COCOA) This is confusing. Maybe just #endif // PLATFORM(COCOA) then #if !PLATFORM(COCOA)? > Source/WebCore/platform/graphics/GraphicsContext3D.h:66 > +#endif // PLATFORM(COCOA) Confusing. > Source/WebCore/platform/graphics/GraphicsContext3D.h:72 > +#else // __OBJC__ Also. > Source/WebCore/platform/graphics/GraphicsContext3D.h:80 > +#else // PLATFORM(COCOA) As well.
Interesting! I feel like there was once a webkit-dev debate about this :) No idea how it was resolved, but your solution sounds reasonable to me.
Created attachment 334292 [details] Patch
Comment on attachment 334292 [details] Patch Clearing flags on attachment: 334292 Committed r228849: <https://trac.webkit.org/changeset/228849>
All reviewed patches have been landed. Closing bug.