Title: [223297] trunk/Source/WebCore
Revision
223297
Author
jer.no...@apple.com
Date
2017-10-13 12:55:47 -0700 (Fri, 13 Oct 2017)

Log Message

One last unreviewed build fix; since the IOSurface APIs don't exist at
all on the simulator, just wrap the entirety of the implementation of
copyImageToPlatformTexture() in a #if USE(IOSURFACE) check.

* platform/graphics/cv/VideoTextureCopierCV.cpp:
(WebCore::VideoTextureCopierCV::copyImageToPlatformTexture):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (223296 => 223297)


--- trunk/Source/WebCore/ChangeLog	2017-10-13 19:04:24 UTC (rev 223296)
+++ trunk/Source/WebCore/ChangeLog	2017-10-13 19:55:47 UTC (rev 223297)
@@ -1,5 +1,14 @@
 2017-10-13  Jer Noble  <jer.no...@apple.com>
 
+        One last unreviewed build fix; since the IOSurface APIs don't exist at
+        all on the simulator, just wrap the entirety of the implementation of
+        copyImageToPlatformTexture() in a #if USE(IOSURFACE) check.
+
+        * platform/graphics/cv/VideoTextureCopierCV.cpp:
+        (WebCore::VideoTextureCopierCV::copyImageToPlatformTexture):
+
+2017-10-13  Jer Noble  <jer.no...@apple.com>
+
         Unreviewed build fix for the previous build fix; use the right PAL path for IOSurfaceSPI.h.
 
         * platform/graphics/cv/VideoTextureCopierCV.cpp:

Modified: trunk/Source/WebCore/platform/graphics/cv/VideoTextureCopierCV.cpp (223296 => 223297)


--- trunk/Source/WebCore/platform/graphics/cv/VideoTextureCopierCV.cpp	2017-10-13 19:04:24 UTC (rev 223296)
+++ trunk/Source/WebCore/platform/graphics/cv/VideoTextureCopierCV.cpp	2017-10-13 19:55:47 UTC (rev 223297)
@@ -502,6 +502,7 @@
     if (auto texture = m_textureCache->textureFromImage(image, outputTarget, level, internalFormat, format, type))
         return copyVideoTextureToPlatformTexture(texture.get(), width, height, outputTexture, outputTarget, level, internalFormat, format, type, premultiplyAlpha, flipY);
 
+#if USE(IOSURFACE)
     // FIXME: This currently only supports '420v' and '420f' pixel formats. Investigate supporting more pixel formats.
     OSType pixelFormat = CVPixelBufferGetPixelFormatType(image);
     if (pixelFormat != kCVPixelFormatType_420YpCbCr8BiPlanarVideoRange && pixelFormat != kCVPixelFormatType_420YpCbCr8BiPlanarFullRange) {
@@ -604,6 +605,9 @@
     m_context->bindTexture(videoTextureTarget, 0);
 
     return true;
+#else
+    return false;
+#endif // USE(IOSURFACE)
 }
 
 bool VideoTextureCopierCV::copyVideoTextureToPlatformTexture(TextureType inputVideoTexture, size_t width, size_t height, Platform3DObject outputTexture, GC3Denum outputTarget, GC3Dint level, GC3Denum internalFormat, GC3Denum format, GC3Denum type, bool premultiplyAlpha, bool flipY)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to