Prototype HTMLVideoElement properties for WebGL texImage2D

third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp

Index: third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
diff --git a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
index defdd0fff2a7287ef759fe52fb6b25ce1b9524ab..b84b911a7a01b0606032eb881b47c668ca19560f 100644
--- a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
+++ b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
@@ -5288,6 +5288,7 @@ void WebGLRenderingContextBase::texImageHelperHTMLVideoElement(
     if (video->copyVideoTextureToPlatformTexture(contextGL(), texture->object(),

[DaleCurtis, 2017/03/17 21:47:52]

Sorry, can you instead add the parameters you want to get to this call? I.e.
copyVideoTextureToPlatformTexture() should now take pointers for width, height,
and timestamp. We want to avoid any method that is not this call from providing
parameters.

[Kai Ninomiya, 2017/03/18 01:08:58]

Hm, adding it to copyVideoTextureToPlatformTexture and texImageImpl wouldn't be
hard (each only has one caller). But I may be missing something here: I think it
would have to also be added to paintCurrentFrame for the other paths, which has
6 callers and would be a little more intrusive than I want for a prototype.

Perhaps this could be implemented only for those 2 upload paths, but that might
not be reliable enough for YouTube's testing. I'm not sure what formats take
those paths on what platforms.

[DaleCurtis, 2017/03/20 17:58:13]

Is there any other way to upload a frame? Also 6 callers isn't that much; plus
if you really want to Blink allows default values, so you can just set the new
parameters as nullptr for optional extraction.

[Ken Russell (switch to Gerrit), 2017/03/20 19:14:14]

Dale, there are three different entry points on HTMLVideoElement used to upload
the current frame to WebGL (as well as for other purposes) -- paintCurrentFrame,
copyVideoTextureToPlatformTexture, and texImageImpl.

Since this is known to be a prototype I think that we should minimize the impact
to the code base, and agree with Kai's decision in this area.

If we are going to plumb this query out to all of the call sites then I think a
struct containing this info should be defined somewhere (presumably
src/third_party/WebKit/public/platform/WebMediaPlayer.h) and we should let the
pointer to that be null, rather than passing three pointers to all of these
APIs.

[DaleCurtis, 2017/03/20 21:59:54]

Yes, this all sounds reasonable to me for the experiment. If we decide to launch
these properties out of experimental the API should change in the way you
suggest (details passed along in struct form at time of upload).

                                                  m_unpackPremultiplyAlpha,
                                                  m_unpackFlipY)) {
+      texture->videoUpdateLastUploaded(video->webMediaPlayer());
       return;
     }
 
@@ -5316,6 +5317,7 @@ void WebGLRenderingContextBase::texImageHelperHTMLVideoElement(
                 texture->object(), internalformat, type, level,
                 m_unpackPremultiplyAlpha, m_unpackFlipY, IntPoint(0, 0),
                 IntRect(0, 0, video->videoWidth(), video->videoHeight()))) {
+          texture->videoUpdateLastUploaded(video->webMediaPlayer());
           return;
         }
       }
@@ -5332,8 +5334,10 @@ void WebGLRenderingContextBase::texImageHelperHTMLVideoElement(
             contextGL(), level, convertTexInternalFormat(internalformat, type),
             format, type, xoffset, yoffset, zoffset, m_unpackFlipY,
             m_unpackPremultiplyAlpha &&
-                m_unpackColorspaceConversion == GL_NONE))
+                m_unpackColorspaceConversion == GL_NONE)) {
+      texture->videoUpdateLastUploaded(video->webMediaPlayer());
       return;
+    }
   }
 
   RefPtr<Image> image = videoFrameToImage(video);
@@ -5344,6 +5348,7 @@ void WebGLRenderingContextBase::texImageHelperHTMLVideoElement(
                WebGLImageConversion::HtmlDomVideo, m_unpackFlipY,
                m_unpackPremultiplyAlpha, sourceImageRect, depth,
                unpackImageHeight);
+  texture->videoUpdateLastUploaded(video->webMediaPlayer());
 }
 
 void WebGLRenderingContextBase::texImageBitmapByGPU(ImageBitmap* bitmap,