Change decoder thread to use new thread function type

Minor updates to VideoReceiveStream:
* Change decoder thread to use new thread function type.
* Reduce the time of when video_receiver_ receives callbacks on the process thread to match with Start/Stop of the decoder.
* Not triggering shutdown unless the thread is running.

BUG=webrtc:7361
Review-Url: https://codereview.webrtc.org/2944033003
Cr-Commit-Position: refs/heads/master@{#18675}
Committed: https://chromium.googlesource.com/external/webrtc/+/c8ece43a6266d8de11b678729e71fa2a16b924d5

[tommi, 2017-06-19 18:17:39]

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

[commit-bot: I haz the power, 2017-06-19 18:17:49]

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-06-19 18:21:57]

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

[commit-bot: I haz the power, 2017-06-19 18:21:58]

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/18355)

[tommi, 2017-06-19 18:38:07]

Description was changed from

==========
Change decoder thread to use new thread function type

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

to

==========
Minor updates to VideoReceiveStream:
* Change decoder thread to use new thread function type.
* Reduce the time of when video_receiver_ receives callbacks on the process
thread to match with Start/Stop of the decoder.
* Not triggering shutdown unless the thread is running.

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

[tommi, 2017-06-19 18:38:08]

tommi@webrtc.org changed reviewers:
+ stefan@webrtc.org

[tommi, 2017-06-19 18:38:44]

git cl format

[tommi, 2017-06-19 19:10:19]

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

[commit-bot: I haz the power, 2017-06-19 19:10: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-06-19 19:30:59]

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

[commit-bot: I haz the power, 2017-06-19 19:31:00]

Dry run: This issue passed the CQ dry run.

[stefan-webrtc, 2017-06-20 07:04:37]

stefan@webrtc.org changed reviewers:
+ philipel@webrtc.org

[stefan-webrtc, 2017-06-20 07:04:37]

LGTM, but please let philipel take a look too.

[philipel, 2017-06-20 09:02:08]

lgtm

[tommi, 2017-06-20 09:42:09]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2017-06-20 09:42:18]

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-06-20 09:44:38]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497951729331760,
"parent_rev": "90242483e3e7b247055b3b4413bd5bb8fcec7de6", "commit_rev":
"c8ece43a6266d8de11b678729e71fa2a16b924d5"}

[commit-bot: I haz the power, 2017-06-20 09:44:42]

Description was changed from

==========
Minor updates to VideoReceiveStream:
* Change decoder thread to use new thread function type.
* Reduce the time of when video_receiver_ receives callbacks on the process
thread to match with Start/Stop of the decoder.
* Not triggering shutdown unless the thread is running.

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

to

==========
Minor updates to VideoReceiveStream:
* Change decoder thread to use new thread function type.
* Reduce the time of when video_receiver_ receives callbacks on the process
thread to match with Start/Stop of the decoder.
* Not triggering shutdown unless the thread is running.

BUG=webrtc:7361

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

[commit-bot: I haz the power, 2017-06-20 09:44:46]

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