Delete VideoCaptureCapability::codecType and related logic.

Delete VideoCaptureCapability::codecType and related logic.

The video_capture module includes remnants of support for cameras
producing encoded frames. However, this seems to be unused, and is
explicitly not supported by VideoCaptureImpl::IncomingFrame.

BUG=None

Review-Url: https://codereview.webrtc.org/2668693008
Cr-Commit-Position: refs/heads/master@{#16732}
Committed: https://chromium.googlesource.com/external/webrtc/+/1e3212216816cce68d873aa3ce5dea87f40414d4

[nisse-webrtc, 2017-02-01 10:26:42]

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

[nisse-webrtc, 2017-02-01 10:26:42]

PTAL. Can we delete this code?

[mflodman, 2017-02-17 11:12:44]

ltgm

[mflodman, 2017-02-17 11:13:00]

Or lgtm :)

[nisse-webrtc, 2017-02-17 11:23:26]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2017-02-17 11:23:37]

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

[commit-bot: I haz the power, 2017-02-17 12:05:59]

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

[commit-bot: I haz the power, 2017-02-17 12:06:00]

Try jobs failed on following builders:
  win_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/18742)

[nisse-webrtc, 2017-02-17 15:14:47]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-02-17 15:14:47]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2668693008/#ps60001 (title:
"Rebased.")

[commit-bot: I haz the power, 2017-02-17 15:14:52]

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

[commit-bot: I haz the power, 2017-02-17 15:31:04]

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

[commit-bot: I haz the power, 2017-02-17 15:31:05]

Try jobs failed on following builders:
  win_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/18764)

[nisse-webrtc, 2017-02-17 15:50:58]

On 2017/02/17 15:31:05, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   win_baremetal on master.tryserver.webrtc (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/18764)

I don't quite see why the win_baremetal bot breaks. My best guess is that it's
the logic in DeviceInfoImpl::GetBestMatchedCapability which was broken.

[nisse-webrtc, 2017-02-20 15:42:43]

On 2017/02/17 15:50:58, nisse-webrtc wrote:
> On 2017/02/17 15:31:05, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   win_baremetal on master.tryserver.webrtc (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/18764)
> 
> I don't quite see why the win_baremetal bot breaks. My best guess is that it's
> the logic in DeviceInfoImpl::GetBestMatchedCapability which was broken.

Redid the changes to GetBestMatchedCapability, it seems I initially cut away too
much.

[nisse-webrtc, 2017-02-20 15:42:52]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-02-20 15:42:52]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2668693008/#ps80001 (title:
"Attempt to fix DeviceInfoImpl::GetBestMatchedCapability.")

[commit-bot: I haz the power, 2017-02-20 15:42:58]

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

[commit-bot: I haz the power, 2017-02-20 17:43:33]

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

[commit-bot: I haz the power, 2017-02-20 17:43:34]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_compile_mips_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build
URL)
  android_compile_x64_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build
URL)
  android_compile_x86_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build
URL)
  android_compile_x86_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build
URL)
  android_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_more_configs on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[nisse-webrtc, 2017-02-21 07:24:57]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2017-02-21 07:25:02]

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

[commit-bot: I haz the power, 2017-02-21 07:27:38]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487661897018040,
"parent_rev": "7e560a1bb741a60f9c6a32953e6f20f5192c7dda", "commit_rev":
"1e3212216816cce68d873aa3ce5dea87f40414d4"}

[commit-bot: I haz the power, 2017-02-21 07:27:42]

Description was changed from

==========
Delete VideoCaptureCapability::codecType and related logic.

The video_capture module includes remnants of support for cameras
producing encoded frames. However, this seems to be unused, and is
explicitly not supported by VideoCaptureImpl::IncomingFrame.

BUG=None
==========

to

==========
Delete VideoCaptureCapability::codecType and related logic.

The video_capture module includes remnants of support for cameras
producing encoded frames. However, this seems to be unused, and is
explicitly not supported by VideoCaptureImpl::IncomingFrame.

BUG=None

Review-Url: https://codereview.webrtc.org/2668693008
Cr-Commit-Position: refs/heads/master@{#16732}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1e3212216816cce68d873aa3c...
==========

[commit-bot: I haz the power, 2017-02-21 07:27:44]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/1e3212216816cce68d873aa3c...