Android: Remove codec thread from MediaCodecVideoDecoder.

Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

BUG=webrtc:7439, chromium:695438

[sakal, 2017-03-28 11:18:56]

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

[commit-bot: I haz the power, 2017-03-28 11:19:00]

Dry run: 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-03-28 11:20:30]

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

[commit-bot: I haz the power, 2017-03-28 11:20:31]

Dry run: Try jobs failed on following builders:
  ios32_sim_ios9_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios32_sim_ios9_dbg/buil...)
  ios_arm64_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_arm64_dbg/builds/18993)
  ios_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_dbg/builds/24279)
  ios_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_rel/builds/23094)

[sakal, 2017-03-28 12:43:01]

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

[commit-bot: I haz the power, 2017-03-28 12:43:04]

Dry run: 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-03-28 12:45:35]

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

[commit-bot: I haz the power, 2017-03-28 12:45:36]

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

[sakal, 2017-03-28 12:50:08]

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

[commit-bot: I haz the power, 2017-03-28 12:50:16]

Dry run: 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-03-28 12:54:44]

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

[commit-bot: I haz the power, 2017-03-28 12:54:45]

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

[sakal, 2017-03-28 13:48:05]

Patchset #1 (id:1) has been deleted

[sakal, 2017-03-28 13:48:13]

Patchset #1 (id:20001) has been deleted

[sakal, 2017-03-28 13:48:21]

Patchset #1 (id:40001) has been deleted

[sakal, 2017-03-28 13:48:27]

Patchset #1 (id:60001) has been deleted

[sakal, 2017-03-28 13:48:33]

Patchset #1 (id:80001) has been deleted

[sakal, 2017-03-28 13:59:32]

Description was changed from

==========
WIP: Test removal of the codec thread.

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

to

==========
Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

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

[sakal, 2017-03-28 13:59:32]

sakal@webrtc.org changed reviewers:
+ magjed@webrtc.org

[sakal, 2017-03-28 13:59:40]

PTAL

[tommi, 2017-03-28 14:35:02]

exciting! :D
Btw, this change might address a performance regression we saw on the nexus 9
bot(s):
https://bugs.chromium.org/p/chromium/issues/detail?id=695438

[tommi, 2017-03-28 14:35:26]

Description was changed from

==========
Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

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

to

==========
Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

BUG=chromium:695438
==========

[magjed_webrtc, 2017-03-28 15:00:16]

lgtm

[tommi, 2017-04-04 10:20:37]

On 2017/03/28 15:00:16, magjed_webrtc wrote:
> lgtm

lgtm - fyi - I've sent this patch to the cq now:
https://codereview.webrtc.org/2764573002

[sakal, 2017-04-04 10:44:09]

Description was changed from

==========
Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

BUG=chromium:695438
==========

to

==========
Android: Remove codec thread from MediaCodecVideoDecoder.

All calls to the decoder will be made on the same thread expect for
release. Calling release on a different thread should be safe because the
decoder thread has already been stopped.

BUG=webrtc:7439, chromium:695438
==========

[tommi, 2017-04-04 10:55:14]

The CQ bit was checked by tommi@webrtc.org

[tommi, 2017-04-04 10:55:15]

The patchset sent to the CQ was uploaded after l-g-t-m from magjed@webrtc.org,
tommi@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2777203003/#ps120001 (title:
"Rebase.")

[commit-bot: I haz the power, 2017-04-04 10:55:19]

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

[sakal, 2017-04-04 10:55:19]

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

[commit-bot: I haz the power, 2017-04-04 10:55:29]

Dry run: 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-04-04 11:16:45]

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

[commit-bot: I haz the power, 2017-04-04 11:16:46]

Dry run: This issue passed the CQ dry run.

[tommi, 2017-04-11 13:09:16]

On 2017/04/04 10:55:19, 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/...

I'll include this change with the other one, as the other one got reverted.  CL
here:
https://codereview.webrtc.org/2811963004/