Prototype HTMLVideoElement properties for WebGL texImage2D

media/blink/webmediaplayer_impl.cc

Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 813a643f7cf0fc1e93183549f5163e3738b9889f..fb496dff3ce72a7acdd98aa5ec0d7a5794df79de 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -702,6 +702,16 @@ void WebMediaPlayerImpl::selectedVideoTrackChanged(
   pipeline_controller_.OnSelectedVideoTrackChanged(selected_video_track_id);
 }
 
+bool WebMediaPlayerImpl::getCurrentFrameInfo(unsigned* width,
+                                             unsigned* height,
+                                             double* timestamp) {
+  scoped_refptr<VideoFrame> video_frame = GetCurrentFrameFromCompositor();

[Ken Russell (switch to Gerrit), 2017/03/15 04:41:50]

As Dale points out, this isn't guaranteed to be correct because the current
frame may have changed since the point where the HTMLVideoElement called into
the WebMediaPlayer.

Instead, I suggest adding a helper method which records the width, height and
timestamp from a passed VideoFrame along the code paths for the three entry
points -- copyVideoTextureToPlatformTexture, paint, and texImageImpl -- and then
have getCurrentFrameInfo return those cached values.

+  *width = video_frame->natural_size().width();
+  *height = video_frame->natural_size().height();
+  *timestamp = video_frame->timestamp().InSecondsF();
+  return true;
+}
+
 blink::WebSize WebMediaPlayerImpl::naturalSize() const {
   DCHECK(main_task_runner_->BelongsToCurrentThread());