[Remoting Host] Add more codec spec information for VP9

[Remoting Host] Add more codec spec information for VP9

The Android hardware decoder can't properly decode the desktop if the codec
spec is not properly set.

This fix is only for VP9 I420. I444 hardware decoding codepath is still broken.

BUG=706220
Review-Url: https://codereview.chromium.org/2837583004
Cr-Commit-Position: refs/heads/master@{#466720}
Committed: https://chromium.googlesource.com/chromium/src/+/7961ed28c4513ca0415a28720ff43a5109794f23

[Yuwei, 2017-04-21 22:36:05]

Description was changed from

==========
[Remoting Host] Add more codec spec information for VP9

The Android client can't properly render the desktop if the codec spec is not
properly set.

This fix is only for VP9 I420. I444 hardware decoding codepath is still broken.

BUG=706220
==========

to

==========
[Remoting Host] Add more codec spec information for VP9

The Android hardware decoder can't properly decode the desktop if the codec
spec is not properly set.

This fix is only for VP9 I420. I444 hardware decoding codepath is still broken.

BUG=706220
==========

[Yuwei, 2017-04-21 22:36:27]

yuweih@chromium.org changed reviewers:
+ sergeyu@chromium.org

[Yuwei, 2017-04-21 22:36:27]

PTAL

[Sergey Ulanov, 2017-04-24 17:11:35]

lgtm

https://codereview.chromium.org/2837583004/diff/1/remoting/protocol/webrtc_du...
File remoting/protocol/webrtc_dummy_video_encoder.cc (right):

https://codereview.chromium.org/2837583004/diff/1/remoting/protocol/webrtc_du...
remoting/protocol/webrtc_dummy_video_encoder.cc:136:
webrtc::CodecSpecificInfoVP9* vp9_info =
maybe add vp8_info for VP8 above, for consistency.

[Yuwei, 2017-04-24 19:13:27]

Thanks!

https://codereview.chromium.org/2837583004/diff/1/remoting/protocol/webrtc_du...
File remoting/protocol/webrtc_dummy_video_encoder.cc (right):

https://codereview.chromium.org/2837583004/diff/1/remoting/protocol/webrtc_du...
remoting/protocol/webrtc_dummy_video_encoder.cc:136:
webrtc::CodecSpecificInfoVP9* vp9_info =
On 2017/04/24 17:11:35, Sergey Ulanov wrote:
> maybe add vp8_info for VP8 above, for consistency.

Done.

[Yuwei, 2017-04-24 19:13:35]

The CQ bit was checked by yuweih@chromium.org

[Yuwei, 2017-04-24 19:13:35]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/2837583004/#ps20001
(title: "Fix Feedback")

[commit-bot: I haz the power, 2017-04-24 19:14:36]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-24 19:44:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493061215462000,
"parent_rev": "29c3639d106664d10b7b42aa4765701dec803b5e", "commit_rev":
"7961ed28c4513ca0415a28720ff43a5109794f23"}

[commit-bot: I haz the power, 2017-04-24 19:44:53]

Description was changed from

==========
[Remoting Host] Add more codec spec information for VP9

The Android hardware decoder can't properly decode the desktop if the codec
spec is not properly set.

This fix is only for VP9 I420. I444 hardware decoding codepath is still broken.

BUG=706220
==========

to

==========
[Remoting Host] Add more codec spec information for VP9

The Android hardware decoder can't properly decode the desktop if the codec
spec is not properly set.

This fix is only for VP9 I420. I444 hardware decoding codepath is still broken.

BUG=706220

Review-Url: https://codereview.chromium.org/2837583004
Cr-Commit-Position: refs/heads/master@{#466720}
Committed:
https://chromium.googlesource.com/chromium/src/+/7961ed28c4513ca0415a28720ff4...
==========

[commit-bot: I haz the power, 2017-04-24 19:44:54]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7961ed28c4513ca0415a28720ff4...