Delete I420 encoder and decoder.

Delete I420 encoder and decoder.

BUG=webrtc:5791

[nisse-webrtc, 2016-04-14 12:05:15]

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

[commit-bot: I haz the power, 2016-04-14 12:05:25]

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

[commit-bot: I haz the power, 2016-04-14 12:07:00]

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

[commit-bot: I haz the power, 2016-04-14 12:07:01]

Dry run: Try jobs failed on following builders:
  win_x64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_rel/builds/13889)

[nisse-webrtc, 2016-04-15 07:46:29]

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

[commit-bot: I haz the power, 2016-04-15 07:46:35]

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

[commit-bot: I haz the power, 2016-04-15 07:54:48]

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

[commit-bot: I haz the power, 2016-04-15 07:54:48]

Dry run: Try jobs failed on following builders:
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/4816)

[nisse-webrtc, 2016-04-15 08:42:37]

Description was changed from

==========
Delete I420 encoder and decoder.

BUG=
==========

to

==========
Delete I420 encoder and decoder.

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

[nisse-webrtc, 2016-04-15 08:50:15]

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

[nisse-webrtc, 2016-04-15 08:50:15]

Some questions:

1. Are any applications likely to use these codecs? Any deprecation dances
needed?

2. If they are deleted, should we aim to delete the  kVideoCodecI420 enum value
as well? (Needs an update in Chrome first).

3. Who should make the decision? etymology: 'from Latin decidere "to decide,
determine," literally "to cut off,"' which is exactly what I'd like t do here
;-)

[pbos-webrtc, 2016-04-15 09:10:29]

I think this should instead of being deleted be moved to
webrtc/test/fake_encoder.{h,cc} and then in the future be wired up to be usable
from video_loopback.

[nisse-webrtc, 2016-04-27 08:36:26]

On 2016/04/15 09:10:29, pbos-webrtc wrote:
> I think this should instead of being deleted be moved to
> webrtc/test/fake_encoder.{h,cc} and then in the future be wired up to be
usable
> from video_loopback.

Some details: Move the I420 code to webrtc/test/, hook into
video_quality_test.cc:SetupCommon.

[pbos-webrtc, 2017-03-16 18:10:07]

pbos@webrtc.org changed reviewers:
- pbos@webrtc.org