⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 273568 in webkit


Ignore:
Timestamp:
Feb 26, 2021, 11:28:35 AM (6 years ago)
Author:
jer.noble@apple.com
Message:

[GPUP] Allow painting of GPUP hosted video from the WebContent process
​https://bugs.webkit.org/show_bug.cgi?id=222461

Reviewed by Eric Carlson.

Source/WebCore:

  • Change the return type of pixelBufferForCurrentTime() to return a RetainPtr<> rather than just a raw pointer.
  • Add utility methods on IOSurface to retrieve an IOSurface from a CVPixelBuffer and to reconstitute a CVPixelBuffer from an IOSurface.
  • platform/cocoa/CoreVideoSoftLink.cpp:
  • platform/cocoa/CoreVideoSoftLink.h:
  • platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::pixelBufferForCurrentTime):

  • platform/graphics/MediaPlayer.h:
  • platform/graphics/MediaPlayerPrivate.h:

(WebCore::MediaPlayerPrivateInterface::pixelBufferForCurrentTime):

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h:
  • platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::pixelBufferForCurrentTime):

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:
  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:

(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::pixelBufferForCurrentTime):

  • platform/graphics/cocoa/IOSurface.h:
  • platform/graphics/cocoa/IOSurface.mm:

(WebCore::IOSurface::createFromPixelBuffer):
(WebCore::IOSurface::createPixelBuffer):

  • platform/graphics/opengl/GraphicsContextGLOpenGL.cpp:

(WebCore::GraphicsContextGLOpenGL::copyTextureFromMedia):

Source/WebKit:

Add Cocoa implementations for nativeImageForCurrentTime() and pixelBufferForCurrentTime().
Use those methods to implement paintCurrentFrameInContext().

  • GPUProcess/media/RemoteMediaPlayerProxy.h:
  • GPUProcess/media/RemoteMediaPlayerProxy.messages.in:
  • GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm:

(WebKit::RemoteMediaPlayerProxy::nativeImageForCurrentTime):
(WebKit::RemoteMediaPlayerProxy::pixelBufferForCurrentTime):

  • WebProcess/GPU/media/MediaPlayerPrivateRemote.cpp:

(WebKit::MediaPlayerPrivateRemote::paint):
(WebKit::MediaPlayerPrivateRemote::paintCurrentFrameInContext):
(WebKit::MediaPlayerPrivateRemote::pixelBufferForCurrentTime):

  • WebProcess/GPU/media/MediaPlayerPrivateRemote.h:
  • WebProcess/GPU/media/cocoa/MediaPlayerPrivateRemoteCocoa.mm:

(WebKit::MediaPlayerPrivateRemote::nativeImageForCurrentTime):
(WebKit::MediaPlayerPrivateRemote::pixelBufferForCurrentTime):

Location:
trunk/Source
Files:
20 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r273564 r273568  
     12021-02-26  Jer Noble  <jer.noble@apple.com>
     2
     3        [GPUP] Allow painting of GPUP hosted video from the WebContent process
     4        https://bugs.webkit.org/show_bug.cgi?id=222461
     5
     6        Reviewed by Eric Carlson.
     7
     8        - Change the return type of pixelBufferForCurrentTime() to return a RetainPtr<>
     9          rather than just a raw pointer.
     10
     11        - Add utility methods on IOSurface to retrieve an IOSurface from a CVPixelBuffer
     12          and to reconstitute a CVPixelBuffer from an IOSurface.
     13
     14        * platform/cocoa/CoreVideoSoftLink.cpp:
     15        * platform/cocoa/CoreVideoSoftLink.h:
     16        * platform/graphics/MediaPlayer.cpp:
     17        (WebCore::MediaPlayer::pixelBufferForCurrentTime):
     18        * platform/graphics/MediaPlayer.h:
     19        * platform/graphics/MediaPlayerPrivate.h:
     20        (WebCore::MediaPlayerPrivateInterface::pixelBufferForCurrentTime):
     21        * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h:
     22        * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
     23        (WebCore::MediaPlayerPrivateAVFoundationObjC::pixelBufferForCurrentTime):
     24        * platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:
     25        * platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
     26        (WebCore::MediaPlayerPrivateMediaSourceAVFObjC::pixelBufferForCurrentTime):
     27        * platform/graphics/cocoa/IOSurface.h:
     28        * platform/graphics/cocoa/IOSurface.mm:
     29        (WebCore::IOSurface::createFromPixelBuffer):
     30        (WebCore::IOSurface::createPixelBuffer):
     31        * platform/graphics/opengl/GraphicsContextGLOpenGL.cpp:
     32        (WebCore::GraphicsContextGLOpenGL::copyTextureFromMedia):
     33
    1342021-02-26  Per Arne  <pvollan@apple.com>
    235

  • trunk/Source/WebCore/platform/cocoa/CoreVideoSoftLink.cpp

    r269371 r273568  
    4747SOFT_LINK_FUNCTION_FOR_SOURCE(WebCore, CoreVideo, CVPixelBufferPoolCreatePixelBuffer, CVReturn, (CFAllocatorRef allocator, CVPixelBufferPoolRef pixelBufferPool, CVPixelBufferRef* pixelBufferOut), (allocator, pixelBufferPool, pixelBufferOut))
    4848SOFT_LINK_FUNCTION_FOR_SOURCE(WebCore, CoreVideo, CVPixelBufferGetIOSurface, IOSurfaceRef, (CVPixelBufferRef pixelBuffer), (pixelBuffer))
     49SOFT_LINK_FUNCTION_FOR_SOURCE(WebCore, CoreVideo, CVImageBufferGetColorSpace, CGColorSpaceRef, (CVImageBufferRef pixelBuffer), (pixelBuffer))
     50
    4951SOFT_LINK_CONSTANT_FOR_SOURCE_WITH_EXPORT(WebCore, CoreVideo, kCVPixelBufferPixelFormatTypeKey, CFStringRef, WEBCORE_EXPORT)
    5052SOFT_LINK_CONSTANT_FOR_SOURCE(WebCore, CoreVideo, kCVPixelBufferCGBitmapContextCompatibilityKey, CFStringRef)

  • trunk/Source/WebCore/platform/cocoa/CoreVideoSoftLink.h

    r265073 r273568  
    6363SOFT_LINK_FUNCTION_FOR_HEADER(WebCore, CoreVideo, CVPixelBufferPoolGetPixelBufferAttributes, CFDictionaryRef, (CVPixelBufferPoolRef pool), (pool))
    6464#define CVPixelBufferPoolGetPixelBufferAttributes softLink_CoreVideo_CVPixelBufferPoolGetPixelBufferAttributes
     65SOFT_LINK_FUNCTION_FOR_HEADER(WebCore, CoreVideo, CVImageBufferGetColorSpace, CGColorSpaceRef, (CVImageBufferRef pixelBuffer), (pixelBuffer))
     66#define CVImageBufferGetColorSpace softLink_CoreVideo_CVImageBufferGetColorSpace
    6567
    6668SOFT_LINK_CONSTANT_FOR_HEADER(WebCore, CoreVideo, kCVPixelBufferPixelFormatTypeKey, CFStringRef)

  • trunk/Source/WebCore/platform/graphics/MediaPlayer.cpp

    r273213 r273568  
    10161016#else
    10171017
    1018 CVPixelBufferRef MediaPlayer::pixelBufferForCurrentTime()
     1018RetainPtr<CVPixelBufferRef> MediaPlayer::pixelBufferForCurrentTime()
    10191019{
    10201020    return m_private->pixelBufferForCurrentTime();

  • trunk/Source/WebCore/platform/graphics/MediaPlayer.h

    r273272 r273568  
    445445    bool copyVideoTextureToPlatformTexture(GraphicsContextGL*, PlatformGLObject texture, GCGLenum target, GCGLint level, GCGLenum internalFormat, GCGLenum format, GCGLenum type, bool premultiplyAlpha, bool flipY);
    446446#else
    447     CVPixelBufferRef pixelBufferForCurrentTime();
     447    RetainPtr<CVPixelBufferRef> pixelBufferForCurrentTime();
    448448#endif
    449449

  • trunk/Source/WebCore/platform/graphics/MediaPlayerPrivate.h

    r273213 r273568  
    165165    virtual bool copyVideoTextureToPlatformTexture(GraphicsContextGL*, PlatformGLObject, GCGLenum, GCGLint, GCGLenum, GCGLenum, GCGLenum, bool, bool) { return false; }
    166166#else
    167     virtual CVPixelBufferRef pixelBufferForCurrentTime() { return nullptr; }
     167    virtual RetainPtr<CVPixelBufferRef> pixelBufferForCurrentTime() { return nullptr; }
    168168#endif
    169169    virtual RefPtr<NativeImage> nativeImageForCurrentTime() { return nullptr; }

  • trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h

    r273213 r273568  
    255255    void waitForVideoOutputMediaDataWillChange();
    256256
    257     CVPixelBufferRef pixelBufferForCurrentTime() final;
     257    RetainPtr<CVPixelBufferRef> pixelBufferForCurrentTime() final;
    258258
    259259#if ENABLE(LEGACY_ENCRYPTED_MEDIA)

  • trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm

    r273498 r273568  
    23922392}
    23932393
    2394 CVPixelBufferRef MediaPlayerPrivateAVFoundationObjC::pixelBufferForCurrentTime()
     2394RetainPtr<CVPixelBufferRef> MediaPlayerPrivateAVFoundationObjC::pixelBufferForCurrentTime()
    23952395{
    23962396    updateLastPixelBuffer();
    … …  
    23982398        return nullptr;
    23992399
    2400     return m_lastPixelBuffer.get();
     2400    return m_lastPixelBuffer;
    24012401}
    24022402

  • trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h

    r273213 r273568  
    222222    void paint(GraphicsContext&, const FloatRect&) override;
    223223    void paintCurrentFrameInContext(GraphicsContext&, const FloatRect&) override;
    224     CVPixelBufferRef pixelBufferForCurrentTime() final;
     224    RetainPtr<CVPixelBufferRef> pixelBufferForCurrentTime() final;
    225225
    226226    bool supportsAcceleratedRendering() const override;

  • trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm

    r273213 r273568  
    624624}
    625625
    626 CVPixelBufferRef MediaPlayerPrivateMediaSourceAVFObjC::pixelBufferForCurrentTime()
     626RetainPtr<CVPixelBufferRef> MediaPlayerPrivateMediaSourceAVFObjC::pixelBufferForCurrentTime()
    627627{
    628628    // We have been asked to paint into a WebGL canvas, so take that as a signal to create
    … …  
    639639    }
    640640
    641     return m_lastPixelBuffer.get();
     641    return m_lastPixelBuffer;
    642642}
    643643

  • trunk/Source/WebCore/platform/graphics/cocoa/IOSurface.h

    r271673 r273568  
    3535#define HAVE_IOSURFACE_RGB10 1
    3636#endif
     37
     38using CVPixelBufferRef = struct __CVBuffer*;
    3739
    3840namespace WTF {
    … …  
    105107    static std::unique_ptr<IOSurface> createFromSurface(IOSurfaceRef, CGColorSpaceRef);
    106108    WEBCORE_EXPORT static std::unique_ptr<IOSurface> createFromImage(CGImageRef);
     109    WEBCORE_EXPORT static std::unique_ptr<IOSurface> createFromPixelBuffer(CVPixelBufferRef);
    107110   
    108111#if USE(IOSURFACE_CANVAS_BACKING_STORE)
    … …  
    123126    WEBCORE_EXPORT RetainPtr<CGImageRef> createImage();
    124127    WEBCORE_EXPORT static RetainPtr<CGImageRef> sinkIntoImage(std::unique_ptr<IOSurface>);
     128    WEBCORE_EXPORT RetainPtr<CVPixelBufferRef> createPixelBuffer();
    125129
    126130#ifdef __OBJC__

  • trunk/Source/WebCore/platform/graphics/cocoa/IOSurface.mm

    r272616 r273568  
    4141#import <wtf/text/TextStream.h>
    4242
     43#import <WebCore/CoreVideoSoftLink.h>
     44
    4345namespace WebCore {
    4446
    … …  
    101103    CGContextFlush(surfaceContext);
    102104    return surface;
     105}
     106
     107std::unique_ptr<IOSurface> IOSurface::createFromPixelBuffer(CVPixelBufferRef pixelBuffer)
     108{
     109    if (!pixelBuffer)
     110        return nullptr;
     111
     112    auto surface = CVPixelBufferGetIOSurface(pixelBuffer);
     113    if (!surface)
     114        return nullptr;
     115
     116    auto colorSpace = CVImageBufferGetColorSpace(pixelBuffer);
     117    if (!colorSpace)
     118        return nullptr;
     119
     120    return createFromSurface(surface, colorSpace);
    103121}
    104122
    … …  
    281299}
    282300
     301RetainPtr<CVPixelBufferRef> IOSurface::createPixelBuffer()
     302{
     303    CVPixelBufferRef rawBuffer = nullptr;
     304    auto status = CVPixelBufferCreateWithIOSurface(kCFAllocatorDefault, m_surface.get(), nullptr, &rawBuffer);
     305    if (status == noErr && rawBuffer)
     306        return adoptCF(rawBuffer);
     307
     308    return nullptr;
     309}
     310
    283311void IOSurface::setContextSize(IntSize contextSize)
    284312{

  • trunk/Source/WebCore/platform/graphics/opengl/GraphicsContextGLOpenGL.cpp

    r273213 r273568  
    272272    UNUSED_VARIABLE(premultiplyAlpha);
    273273    ASSERT_UNUSED(outputTarget, outputTarget == GraphicsContextGL::TEXTURE_2D);
    274     return contextCV->copyPixelBufferToTexture(pixelBuffer, outputTexture, level, internalFormat, format, type, GraphicsContextGL::FlipY(flipY));
     274    return contextCV->copyPixelBufferToTexture(pixelBuffer.get(), outputTexture, level, internalFormat, format, type, GraphicsContextGL::FlipY(flipY));
    275275#else
    276276    return player.copyVideoTextureToPlatformTexture(this, outputTexture, outputTarget, level, internalFormat, format, type, premultiplyAlpha, flipY);

  • trunk/Source/WebKit/ChangeLog

    r273564 r273568  
     12021-02-26  Jer Noble  <jer.noble@apple.com>
     2
     3        [GPUP] Allow painting of GPUP hosted video from the WebContent process
     4        https://bugs.webkit.org/show_bug.cgi?id=222461
     5
     6        Reviewed by Eric Carlson.
     7
     8        Add Cocoa implementations for nativeImageForCurrentTime() and pixelBufferForCurrentTime().
     9        Use those methods to implement paintCurrentFrameInContext().
     10
     11        * GPUProcess/media/RemoteMediaPlayerProxy.h:
     12        * GPUProcess/media/RemoteMediaPlayerProxy.messages.in:
     13        * GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm:
     14        (WebKit::RemoteMediaPlayerProxy::nativeImageForCurrentTime):
     15        (WebKit::RemoteMediaPlayerProxy::pixelBufferForCurrentTime):
     16        * WebProcess/GPU/media/MediaPlayerPrivateRemote.cpp:
     17        (WebKit::MediaPlayerPrivateRemote::paint):
     18        (WebKit::MediaPlayerPrivateRemote::paintCurrentFrameInContext):
     19        (WebKit::MediaPlayerPrivateRemote::pixelBufferForCurrentTime):
     20        * WebProcess/GPU/media/MediaPlayerPrivateRemote.h:
     21        * WebProcess/GPU/media/cocoa/MediaPlayerPrivateRemoteCocoa.mm:
     22        (WebKit::MediaPlayerPrivateRemote::nativeImageForCurrentTime):
     23        (WebKit::MediaPlayerPrivateRemote::pixelBufferForCurrentTime):
     24
    1252021-02-26  Per Arne  <pvollan@apple.com>
    226

  • trunk/Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.h

    r273473 r273568  
    287287    void setShouldEnableAudioSourceProvider(bool);
    288288
     289#if PLATFORM(COCOA)
     290    void nativeImageForCurrentTime(CompletionHandler<void(Optional<WTF::MachSendRight>&&)>&&);
     291    void pixelBufferForCurrentTime(CompletionHandler<void(Optional<WTF::MachSendRight>&&)>&&);
     292#endif
     293
    289294#if !RELEASE_LOG_DISABLED
    290295    const Logger& mediaPlayerLogger() final { return m_logger; }

  • trunk/Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.messages.in

    r272600 r273568  
    127127    SetShouldEnableAudioSourceProvider(bool shouldEnable)
    128128#endif
     129
     130#if PLATFORM(COCOA)
     131    NativeImageForCurrentTime() -> (Optional<MachSendRight> sendRight) Synchronous
     132    PixelBufferForCurrentTime() -> (Optional<MachSendRight> sendRight) Synchronous
     133#endif
    129134}
    130135

  • trunk/Source/WebKit/GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm

    r272414 r273568  
    3232#import "MediaPlayerPrivateRemoteMessages.h"
    3333#import <QuartzCore/QuartzCore.h>
     34#import <WebCore/IOSurface.h>
    3435#import <WebCore/IntSize.h>
    3536#import <wtf/MachSendRight.h>
    … …  
    7778}
    7879
     80void RemoteMediaPlayerProxy::nativeImageForCurrentTime(CompletionHandler<void(Optional<WTF::MachSendRight>&&)>&& completionHandler)
     81{
     82    if (!m_player) {
     83        completionHandler(WTF::nullopt);
     84        return;
     85    }
     86
     87    auto nativeImage = m_player->nativeImageForCurrentTime();
     88    if (!nativeImage) {
     89        completionHandler(WTF::nullopt);
     90        return;
     91    }
     92
     93    auto platformImage = nativeImage->platformImage();
     94    if (!platformImage) {
     95        completionHandler(WTF::nullopt);
     96        return;
     97    }
     98
     99    auto surface = WebCore::IOSurface::createFromImage(platformImage.get());
     100    if (!surface) {
     101        completionHandler(WTF::nullopt);
     102        return;
     103    }
     104
     105    completionHandler(surface->createSendRight());
     106}
     107
     108void RemoteMediaPlayerProxy::pixelBufferForCurrentTime(CompletionHandler<void(Optional<WTF::MachSendRight>&&)>&& completionHandler)
     109{
     110#if !USE(AVFOUNDATION)
     111    completionHandler(WTF::nullopt);
     112#else
     113    if (!m_player) {
     114        completionHandler(WTF::nullopt);
     115        return;
     116    }
     117
     118    auto pixelBuffer = m_player->pixelBufferForCurrentTime();
     119    if (!pixelBuffer) {
     120        completionHandler(WTF::nullopt);
     121        return;
     122    }
     123
     124    auto surface = WebCore::IOSurface::createFromPixelBuffer(pixelBuffer.get());
     125    if (!surface) {
     126        completionHandler(WTF::nullopt);
     127        return;
     128    }
     129
     130    completionHandler(surface->createSendRight());
     131#endif
     132}
     133
    79134} // namespace WebKit
    80135

  • trunk/Source/WebKit/WebProcess/GPU/media/MediaPlayerPrivateRemote.cpp

    r273213 r273568  
    4343#include <JavaScriptCore/GenericTypedArrayViewInlines.h>
    4444#include <JavaScriptCore/TypedArrayType.h>
     45#include <WebCore/GraphicsContext.h>
    4546#include <WebCore/MediaPlayer.h>
    4647#include <WebCore/NotImplemented.h>
    … …  
    839840#endif
    840841
    841 void MediaPlayerPrivateRemote::paint(GraphicsContext&, const FloatRect&)
    842 {
    843     notImplemented();
    844 }
    845 
    846 void MediaPlayerPrivateRemote::paintCurrentFrameInContext(GraphicsContext&, const FloatRect&)
    847 {
    848     notImplemented();
     842void MediaPlayerPrivateRemote::paint(GraphicsContext& context, const FloatRect& rect)
     843{
     844    paintCurrentFrameInContext(context, rect);
     845}
     846
     847void MediaPlayerPrivateRemote::paintCurrentFrameInContext(GraphicsContext& context, const FloatRect& rect)
     848{
     849    if (context.paintingDisabled())
     850        return;
     851
     852    auto nativeImage = nativeImageForCurrentTime();
     853    if (!nativeImage)
     854        return;
     855
     856    FloatRect imageRect { FloatPoint::zero(), nativeImage->size() };
     857    context.drawNativeImage(*nativeImage, imageRect.size(), rect, imageRect);
    849858}
    850859
    … …  
    855864    return false;
    856865}
    857 #endif
    858 
     866#elif !PLATFORM(COCOA)
     867RetainPtr<CVPixelBufferRef> MediaPlayerPrivateRemote::pixelBufferForCurrentTime()
     868{
     869    notImplemented();
     870    return false;
     871}
     872#endif
     873
     874#if !PLATFORM(COCOA)
    859875RefPtr<NativeImage> MediaPlayerPrivateRemote::nativeImageForCurrentTime()
    860876{
    … …  
    862878    return nullptr;
    863879}
     880#endif
    864881
    865882bool MediaPlayerPrivateRemote::hasAvailableVideoFrame() const

  • trunk/Source/WebKit/WebProcess/GPU/media/MediaPlayerPrivateRemote.h

    r273213 r273568  
    284284#if !USE(AVFOUNDATION)
    285285    bool copyVideoTextureToPlatformTexture(WebCore::GraphicsContextGL*, PlatformGLObject, GCGLenum, GCGLint, GCGLenum, GCGLenum, GCGLenum, bool, bool) final;
     286#else
     287    RetainPtr<CVPixelBufferRef> pixelBufferForCurrentTime() final;
    286288#endif
    287289    RefPtr<WebCore::NativeImage> nativeImageForCurrentTime() final;

  • trunk/Source/WebKit/WebProcess/GPU/media/cocoa/MediaPlayerPrivateRemoteCocoa.mm

    r268521 r273568  
    3030
    3131#import "RemoteAudioSourceProvider.h"
     32#import "RemoteMediaPlayerProxyMessages.h"
     33#import "WebCoreArgumentCoders.h"
     34#import <WebCore/ColorSpaceCG.h>
     35#import <WebCore/IOSurface.h>
    3236#import <WebCore/VideoLayerManagerObjC.h>
    3337#import <pal/spi/cocoa/QuartzCoreSPI.h>
     38#import <wtf/MachSendRight.h>
    3439
    3540namespace WebKit {
    … …  
    5863#endif
    5964
     65RefPtr<NativeImage> MediaPlayerPrivateRemote::nativeImageForCurrentTime()
     66{
     67    Optional<MachSendRight> sendRight;
     68    if (!connection().sendSync(Messages::RemoteMediaPlayerProxy::NativeImageForCurrentTime(), Messages::RemoteMediaPlayerProxy::NativeImageForCurrentTime::Reply(sendRight), m_id))
     69        return nullptr;
     70
     71    if (!sendRight)
     72        return nullptr;
     73
     74    auto surface = WebCore::IOSurface::createFromSendRight(WTFMove(*sendRight), sRGBColorSpaceRef());
     75    if (!surface)
     76        return nullptr;
     77
     78    auto platformImage = WebCore::IOSurface::sinkIntoImage(WTFMove(surface));
     79    if (!platformImage)
     80        return nullptr;
     81
     82    return NativeImage::create(WTFMove(platformImage));
     83}
     84
     85RetainPtr<CVPixelBufferRef> MediaPlayerPrivateRemote::pixelBufferForCurrentTime()
     86{
     87    Optional<MachSendRight> sendRight;
     88    if (!connection().sendSync(Messages::RemoteMediaPlayerProxy::PixelBufferForCurrentTime(), Messages::RemoteMediaPlayerProxy::NativeImageForCurrentTime::Reply(sendRight), m_id))
     89        return nullptr;
     90
     91    if (!sendRight)
     92        return nullptr;
     93
     94    auto surface = WebCore::IOSurface::createFromSendRight(WTFMove(*sendRight), nullptr);
     95    if (!surface)
     96        return nullptr;
     97
     98    return surface->createPixelBuffer();
     99}
     100
    60101} // namespace WebKit
    61102
Note: See TracChangeset for help on using the changeset viewer.