Memory copy the VideoFrame to match the requirement for HW encoders.

media/base/video_util.h

Index: media/base/video_util.h
diff --git a/media/base/video_util.h b/media/base/video_util.h
index 2dec7e9371013ff3bf0442cb798cef0c90104659..a7fad72ff182f68883cce51a7c310b032abe4ed3 100644
--- a/media/base/video_util.h
+++ b/media/base/video_util.h
@@ -97,6 +97,11 @@ MEDIA_EXPORT void CopyRGBToVideoFrame(const uint8_t* source,
 MEDIA_EXPORT scoped_refptr<VideoFrame> WrapAsI420VideoFrame(
     const scoped_refptr<VideoFrame>& frame);
 
+// Copy I420 video frame to match the required coded size and pad it repeatly
+// with the last column / row.

[miu, 2016/04/30 00:41:14]

Comment should also mention: 1) The visible region in both frames should be
identical; 2) When/why would this function would return true or false.

[xjz, 2016/05/03 01:17:58]

Done.

+MEDIA_EXPORT bool I420CopyWithPadding(const scoped_refptr<VideoFrame> src_frame,

[miu, 2016/04/30 00:41:14]

Type of the args should be:

const VideoFrame& src_frame,  // because it's read-only and a non-null frame is
required
VideoFrame* dst_frame,  // because nothing will take ownership of dst_frame

[xjz, 2016/05/03 01:17:58]

Done.

+                                      scoped_refptr<VideoFrame> dst_frame);

[miu, 2016/04/30 00:41:14]

Consider adding WARN_UNUSED_RESULT in the function signature. This will force
callers of this function to examine the return boolean. Meaning:

  MEDIA_EXPORT bool I420CopyWithPadding(...) WARN_UNUSED_RESULT;

[xjz, 2016/05/03 01:17:59]

Done.

+
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_UTIL_H_