Enable and remove 2 tests in WebRtcVideoChannel2Test

WebRtcVideoChannel2Test::SetRecvCodecsAcceptsMultipleVideoCodecs passes now.
WebRtcVideoChannel2Test::SetRecvCodecsSetsFecForAllVideoCodecs was never
fully implemented and hasn't been touched in over a year.

BUG=NONE

Review-Url: https://codereview.webrtc.org/2736483002
Cr-Commit-Position: refs/heads/master@{#17075}
Committed: https://chromium.googlesource.com/external/webrtc/+/d5a2d9ad0cbfa38f58582c19920c24733059d2a2

[Zach Stein, 2017-03-04 06:39:18]

Description was changed from

==========
Enabling a disabled test that passes.

BUG=webrtc:
==========

to

==========
Enabling a disabled test that passes and removing an unimplemented test that
hasn't been touched for over a year.

BUG=webrtc:
==========

[Zach Stein, 2017-03-04 07:01:48]

zstein@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[Taylor Brandstetter, 2017-03-06 00:36:06]

lgtm

For something small like this, you can use BUG=None. Also, try to fit the
description to 80 columns wide.

Also, I don't think I'm an owner of this directory.

[Zach Stein, 2017-03-06 19:26:16]

Description was changed from

==========
Enabling a disabled test that passes and removing an unimplemented test that
hasn't been touched for over a year.

BUG=webrtc:
==========

to

==========
Enabling a disabled test that passes and removing an unimplemented test
that hasn't been touched for over a year.

BUG=NONE
==========

[Zach Stein, 2017-03-06 19:30:07]

zstein@webrtc.org changed reviewers:
+ pbos@webrtc.org

[Zach Stein, 2017-03-06 19:30:08]

Could you take a quick look pbos? Thanks!

[pbos-webrtc, 2017-03-06 19:32:36]

lgtm, but please update the CL title + description to say which test you're
disabling.

"Enabling a disabled test that passes." is way too broad. Keep the title +
description informative.

[Zach Stein, 2017-03-06 19:37:41]

Description was changed from

==========
Enabling a disabled test that passes and removing an unimplemented test
that hasn't been touched for over a year.

BUG=NONE
==========

to

==========
WebRtcVideoChannel2Test::SetRecvCodecsAcceptsMultipleVideoCodecs passes now.
WebRtcVideoChannel2Test::SetRecvCodecsSetsFecForAllVideoCodecs was never
fully implemented and hasn't been touched in over a year.

BUG=NONE
==========

[Zach Stein, 2017-03-06 19:38:47]

The CQ bit was checked by zstein@webrtc.org

[commit-bot: I haz the power, 2017-03-06 19:38:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[pbos-webrtc, 2017-03-06 19:41:36]

On 2017/03/06 19:38:53, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...


The title of the issue should be the same as the first line of the CL
description. The CL description is what goes in as commit message. Keep the
one-line description in both the title + description.

"Enable and remove 2 tests" is misleading. Suggesting "Re-enable
SetRecvCodecsAcceptsMultipleVideoCodecs test." as header, then in the
description say "Also removing SetRecvCodecsSetsFecForAllVideoCodecs which
hasn't been implemented or touched for over a year." as well.

[commit-bot: I haz the power, 2017-03-06 20:09:25]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488829127325930,
"parent_rev": "eaa9c1db73133d771ea7a9b7ceb39b4286df0e78", "commit_rev":
"d5a2d9ad0cbfa38f58582c19920c24733059d2a2"}

[commit-bot: I haz the power, 2017-03-06 20:09:27]

Description was changed from

==========
WebRtcVideoChannel2Test::SetRecvCodecsAcceptsMultipleVideoCodecs passes now.
WebRtcVideoChannel2Test::SetRecvCodecsSetsFecForAllVideoCodecs was never
fully implemented and hasn't been touched in over a year.

BUG=NONE
==========

to

==========
WebRtcVideoChannel2Test::SetRecvCodecsAcceptsMultipleVideoCodecs passes now.
WebRtcVideoChannel2Test::SetRecvCodecsSetsFecForAllVideoCodecs was never
fully implemented and hasn't been touched in over a year.

BUG=NONE

Review-Url: https://codereview.webrtc.org/2736483002
Cr-Commit-Position: refs/heads/master@{#17075}
Committed:
https://chromium.googlesource.com/external/webrtc/+/d5a2d9ad0cbfa38f58582c199...
==========

[commit-bot: I haz the power, 2017-03-06 20:09:28]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/d5a2d9ad0cbfa38f58582c199...