Add native-handle support for single VP8 streams.

Add native-handle support for single VP8 streams.

Implements SupportsNativeHandle() in SimulcastEncoderAdapter which works
when there's only a single encoder.

BUG=webrtc:5060
R=stefan@webrtc.org

Committed: https://crrev.com/65e15bafaaf18a5361a9831b17254810d5cb3111
Cr-Commit-Position: refs/heads/master@{#10291}

[pbos-webrtc, 2015-10-09 13:54:10]

PTAL, woo!

[pbos-webrtc, 2015-10-09 13:54:40]

The CQ bit was checked by pbos@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-09 13:54:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1397653004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1397653004/1

[commit-bot: I haz the power, 2015-10-09 14:36:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-09 14:36:49]

Dry run: This issue passed the CQ dry run.

[stefan-webrtc, 2015-10-13 15:07:52]

lgtm

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
File
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc
(right):

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:141:
bool SupportsNativeHandle() const override {
supports_native_handle

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:160:
void SetSupportsNativeHandle(bool enabled) {
set_supports_native_handle

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:405:
EXPECT_FALSE(adapter_->SupportsNativeHandle());
Add a case for encoders()[1] too?

[pbos-webrtc, 2015-10-15 16:42:47]

feedback

[pbos-webrtc, 2015-10-15 16:43:34]

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
File
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc
(right):

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:141:
bool SupportsNativeHandle() const override {
On 2015/10/13 15:07:51, stefan-webrtc (holmer) wrote:
> supports_native_handle

Nope, inherited.

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:160:
void SetSupportsNativeHandle(bool enabled) {
On 2015/10/13 15:07:52, stefan-webrtc (holmer) wrote:
> set_supports_native_handle

Done.

https://codereview.webrtc.org/1397653004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter_unittest.cc:405:
EXPECT_FALSE(adapter_->SupportsNativeHandle());
On 2015/10/13 15:07:51, stefan-webrtc (holmer) wrote:
> Add a case for encoders()[1] too?

Done.

[pbos-webrtc, 2015-10-15 16:43:49]

The CQ bit was checked by pbos@webrtc.org

[pbos-webrtc, 2015-10-15 16:43:49]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1397653004/#ps20001 (title:
"feedback")

[commit-bot: I haz the power, 2015-10-15 16:43:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1397653004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1397653004/20001

[commit-bot: I haz the power, 2015-10-15 17:52:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-15 17:52:31]

Patchset 2 (id:??) landed as
https://crrev.com/65e15bafaaf18a5361a9831b17254810d5cb3111
Cr-Commit-Position: refs/heads/master@{#10291}