16-bit video upload to float: intermediate R16_EXT and copy to float.

media/renderers/skcanvas_video_renderer.h

Index: media/renderers/skcanvas_video_renderer.h
diff --git a/media/renderers/skcanvas_video_renderer.h b/media/renderers/skcanvas_video_renderer.h
index 3831139c03cc49e8c1fbe1c687daa191921cb9eb..fa091eed6a2542da854e52ac4c428bd6daec22fb 100644
--- a/media/renderers/skcanvas_video_renderer.h
+++ b/media/renderers/skcanvas_video_renderer.h
@@ -28,6 +28,10 @@ namespace gfx {
 class RectF;
 }
 
+namespace gpu {
+struct Capabilities;
+}
+
 namespace media {
 
 // TODO(enne): rename to PaintCanvasVideoRenderer
@@ -100,7 +104,9 @@ class MEDIA_EXPORT SkCanvasVideoRenderer {
   // parameters.
   // Returns false if there is no implementation for given parameters.
   static bool TexImage2D(unsigned target,
+                         unsigned texture,

[Ken Russell (switch to Gerrit), 2017/04/20 01:47:07]

Please document |texture| -- i.e., if it's the one currently bound to |target|,
what is the state of the binding to |target| upon return, etc.

Also, if this is really only used for 16-bit textures, then "TexImage2D" is too
generic and confusing a name, and I'd ask that you rename it to something like
"TexImage2DY16" and change the call sites.

[aleksandar.stojiljkovic, 2017/04/26 13:29:40]

Done.
Plan to use it at least for Y8 and will try to optimize HDR RGB10_A2 upload
using it. Changed the description to clarify it.

                          gpu::gles2::GLES2Interface* gl,
+                         const gpu::Capabilities& gpu_capabilities,
                          VideoFrame* video_frame,
                          int level,
                          int internalformat,