Index: content/renderer/media/webmediaplayer_impl.cc |
diff --git a/content/renderer/media/webmediaplayer_impl.cc b/content/renderer/media/webmediaplayer_impl.cc |
index 91dc4f1b9bf24c82eedde9668e0fc348b0e512ff..4aee9e15c2f66d5437eb1fbaa6d4097051a196c2 100644 |
--- a/content/renderer/media/webmediaplayer_impl.cc |
+++ b/content/renderer/media/webmediaplayer_impl.cc |
@@ -30,8 +30,7 @@ |
#include "content/renderer/media/webmediaplayer_params.h" |
#include "content/renderer/media/webmediaplayer_util.h" |
#include "content/renderer/media/webmediasource_impl.h" |
-#include "gpu/GLES2/gl2extchromium.h" |
-#include "gpu/command_buffer/common/mailbox_holder.h" |
+#include "content/renderer/render_thread_impl.h" |
#include "media/audio/null_audio_sink.h" |
#include "media/base/audio_hardware_config.h" |
#include "media/base/bind_to_current_loop.h" |
@@ -59,6 +58,8 @@ |
#include "third_party/WebKit/public/web/WebLocalFrame.h" |
#include "third_party/WebKit/public/web/WebSecurityOrigin.h" |
#include "third_party/WebKit/public/web/WebView.h" |
+#include "webkit/common/gpu/context_provider_web_context.h" |
+#include "webkit/common/gpu/webgraphicscontext3d_impl.h" |
using blink::WebCanvas; |
using blink::WebMediaPlayer; |
@@ -88,23 +89,6 @@ namespace { |
const double kMinRate = 0.0625; |
const double kMaxRate = 16.0; |
-class SyncPointClientImpl : public media::VideoFrame::SyncPointClient { |
- public: |
- explicit SyncPointClientImpl( |
- blink::WebGraphicsContext3D* web_graphics_context) |
- : web_graphics_context_(web_graphics_context) {} |
- virtual ~SyncPointClientImpl() {} |
- virtual uint32 InsertSyncPoint() OVERRIDE { |
- return web_graphics_context_->insertSyncPoint(); |
- } |
- virtual void WaitSyncPoint(uint32 sync_point) OVERRIDE { |
- web_graphics_context_->waitSyncPoint(sync_point); |
- } |
- |
- private: |
- blink::WebGraphicsContext3D* web_graphics_context_; |
-}; |
- |
} // namespace |
namespace content { |
@@ -513,13 +497,17 @@ void WebMediaPlayerImpl::paint(blink::WebCanvas* canvas, |
GetCurrentFrameFromCompositor(); |
gfx::Rect gfx_rect(rect); |
- |
+ scoped_refptr<cc::ContextProvider> context_provider = |
+ RenderThreadImpl::current()->SharedMainThreadContextProvider(); |
+ media::SkCanvasVideoRenderer::Context3DProvider provider{ |
+ context_provider->ContextGL(), context_provider->GrContext()}; |
skcanvas_video_renderer_.Paint(video_frame.get(), |
canvas, |
gfx_rect, |
alpha, |
mode, |
- pipeline_metadata_.video_rotation); |
+ pipeline_metadata_.video_rotation, |
+ &provider); |
} |
bool WebMediaPlayerImpl::hasSingleSecurityOrigin() const { |
@@ -579,43 +567,23 @@ bool WebMediaPlayerImpl::copyVideoTextureToPlatformTexture( |
scoped_refptr<media::VideoFrame> video_frame = |
GetCurrentFrameFromCompositor(); |
- if (!video_frame.get()) |
- return false; |
- if (video_frame->format() != media::VideoFrame::NATIVE_TEXTURE) |
- return false; |
- |
- const gpu::MailboxHolder* mailbox_holder = video_frame->mailbox_holder(); |
- if (mailbox_holder->texture_target != GL_TEXTURE_2D) |
+ if (!video_frame || |
rileya (GONE FROM CHROMIUM)
2014/09/02 21:21:43
Multi-line conditional needs {}'s.
|
+ video_frame->format() != media::VideoFrame::NATIVE_TEXTURE) |
return false; |
- web_graphics_context->waitSyncPoint(mailbox_holder->sync_point); |
- uint32 source_texture = web_graphics_context->createAndConsumeTextureCHROMIUM( |
- GL_TEXTURE_2D, mailbox_holder->mailbox.name); |
- |
- // The video is stored in a unmultiplied format, so premultiply |
- // if necessary. |
- web_graphics_context->pixelStorei(GL_UNPACK_PREMULTIPLY_ALPHA_CHROMIUM, |
- premultiply_alpha); |
- // Application itself needs to take care of setting the right flip_y |
- // value down to get the expected result. |
- // flip_y==true means to reverse the video orientation while |
- // flip_y==false means to keep the intrinsic orientation. |
- web_graphics_context->pixelStorei(GL_UNPACK_FLIP_Y_CHROMIUM, flip_y); |
- web_graphics_context->copyTextureCHROMIUM(GL_TEXTURE_2D, |
- source_texture, |
- texture, |
- level, |
- internal_format, |
- type); |
- web_graphics_context->pixelStorei(GL_UNPACK_FLIP_Y_CHROMIUM, false); |
- web_graphics_context->pixelStorei(GL_UNPACK_PREMULTIPLY_ALPHA_CHROMIUM, |
- false); |
- |
- web_graphics_context->deleteTexture(source_texture); |
- web_graphics_context->flush(); |
- |
- SyncPointClientImpl client(web_graphics_context); |
- video_frame->UpdateReleaseSyncPoint(&client); |
+ // TODO(dshwang): need more elegant way to convert WebGraphicsContext3D to |
+ // GLES2Interface. |
+ gpu::gles2::GLES2Interface* gl = |
+ static_cast<webkit::gpu::WebGraphicsContext3DImpl*>(web_graphics_context) |
+ ->GetGLInterface(); |
+ skcanvas_video_renderer_.CopyVideoFrameToTexture(gl, |
+ video_frame.get(), |
+ texture, |
+ level, |
+ internal_format, |
+ type, |
+ premultiply_alpha, |
+ flip_y); |
return true; |
} |