V4L2SVDA/VAAPIVDA: use visible size from decoder and pass to client

media/gpu/vaapi_video_decode_accelerator.h

Index: media/gpu/vaapi_video_decode_accelerator.h
diff --git a/media/gpu/vaapi_video_decode_accelerator.h b/media/gpu/vaapi_video_decode_accelerator.h
index 241c002d9c5aa9230d019bbe54e36f95814e01bf..2168727e108cf57b81cae0d1ab3f02ebbf9f59ce 100644
--- a/media/gpu/vaapi_video_decode_accelerator.h
+++ b/media/gpu/vaapi_video_decode_accelerator.h
@@ -145,10 +145,11 @@ class MEDIA_GPU_EXPORT VaapiVideoDecodeAccelerator
 
   // Callback to be executed once we have a |va_surface| to be output and
   // an available |picture| to use for output.
-  // Puts contents of |va_surface| into given |picture|, releases the
-  // surface and passes the resulting picture to client for output.
+  // Puts contents of |va_surface| and set |visible_rect| into given |picture|,

[Pawel Osciak, 2017/06/19 06:27:40]

visible_rect is not a part of picture. Could we perhaps say something like:

"Puts contents of |va_surface| into given |picture|, releases the surface and
passes the resulting picture to client to output the given |visible_rect| part
of it"

?

[johnylin1, 2017/06/19 07:13:28]

Done.

+  // releases the surface and passes the resulting picture to client for output.
   void OutputPicture(const scoped_refptr<VASurface>& va_surface,
                      int32_t input_id,
+                     gfx::Rect visible_rect,
                      VaapiPicture* picture);
 
   // Try to OutputPicture() if we have both a ready surface and picture.