VP9: Add automaticeResize to codec setting.

VP9: Add automaticeResize to codec setting.

Added unittest.
This setting allows for dynamic resizing at low bitrates.
Setting is off by default for now.

BUG=
R=mflodman@webrtc.org, stefan@webrtc.org
TBR=stefan@webrtc.org

Committed: https://crrev.com/f35072019be4e095b079c99be94145a237ecbe88
Cr-Commit-Position: refs/heads/master@{#9976}

[marpan, 2015-09-01 15:50:19]

marpan@google.com changed reviewers:
+ stefan@webrtc.org

[marpan, 2015-09-01 16:27:15]

marpan@google.com changed reviewers:
+ mflodman@webrtc.org

[mflodman, 2015-09-03 12:06:44]

On 2015/09/01 16:27:15, marpan wrote:
> mailto:marpan@google.com changed reviewers:
> + mailto:mflodman@webrtc.org

LGTM.

Is this ready to be enabled in webrtcvideoengine2.cc,
WebRtcVideoChannel2::WebRtcVideoSendStream::ConfigureVideoEncoderSettings, so
that should be done at the same time?

[stefan-webrtc, 2015-09-03 12:27:53]

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc
(right):

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:732:
// |spatial_resizei| is on, so expect one resize during the sequence,
spatial_resize

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:754:
rc_metrics);
Is there any way to actually verify that only the first frame is a key frame?

[mflodman, 2015-09-03 12:40:27]

Hmm, my comment never made it in my previous reply. For some reason.

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc
(right):

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:732:
// |spatial_resizei| is on, so expect one resize during the sequence,
Remove 'i' in "spatial_resizei".

[marpan, 2015-09-04 21:45:45]

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc
(right):

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:732:
// |spatial_resizei| is on, so expect one resize during the sequence,
On 2015/09/03 12:40:26, mflodman wrote:
> Remove 'i' in "spatial_resizei".

Done.

https://codereview.webrtc.org/1323943007/diff/20001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:754:
rc_metrics);
On 2015/09/03 12:27:52, stefan-webrtc (holmer) wrote:
> Is there any way to actually verify that only the first frame is a key frame?
Done. Added that for all the tests.

[stefan-webrtc, 2015-09-05 10:45:49]

Great improvement to the tests. LGTM

https://codereview.webrtc.org/1323943007/diff/40001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc
(right):

https://codereview.webrtc.org/1323943007/diff/40001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc:453:
VideoFrameType frame_type = kDeltaFrame;
I would move this declaration into the loop instead as it isn't used outside.

[marpan, 2015-09-17 19:16:22]

Committed patchset #3 (id:40001) manually as
f35072019be4e095b079c99be94145a237ecbe88 (presubmit successful).

[commit-bot: I haz the power, 2015-09-17 19:16:28]

Patchset 3 (id:??) landed as
https://crrev.com/f35072019be4e095b079c99be94145a237ecbe88
Cr-Commit-Position: refs/heads/master@{#9976}