HW Video: Make media::VideoFrame handle the sync point of the compositor as well as webgl

content/renderer/media/webmediaplayer_impl.cc

Index: content/renderer/media/webmediaplayer_impl.cc
diff --git a/content/renderer/media/webmediaplayer_impl.cc b/content/renderer/media/webmediaplayer_impl.cc
index 9399476dd837e07a3435c0bdfebf7ced6fb91742..d8bc2e03af305d0c8ed2741aa15ab84e90a3a854 100644
--- a/content/renderer/media/webmediaplayer_impl.cc
+++ b/content/renderer/media/webmediaplayer_impl.cc
@@ -684,6 +684,7 @@ bool WebMediaPlayerImpl::copyVideoTextureToPlatformTexture(
   // The flush() operation is not necessary here. It is kept since the
   // performance will be better when it is added than not.
   web_graphics_context->flush();

[danakj, 2014/04/11 16:23:28]

I wonder if this flush was trying to deal with this issue, or what "performance"
it means now. Maybe you can revisit this flush afterwards?

[dshwang, 2014/04/11 17:55:37]

This flush is logically needed because we want to make sure gpu driver execute
WebGL's all operations when GPU video decoder context waits for below sync
point.

+  video_frame->AppendReleaseSyncPoint(web_graphics_context->insertSyncPoint());

[dshwang, 2014/03/20 08:53:24]

sievers@, I didnot add this line to webmediaplayer_android.cc, because WMPA uses
one stream texture, so it is not needed.

   return true;
 }