Switch to use SequencedTaskChecker instead of ThreadChecker where needed.

Switch to use SequencedTaskChecker instead of ThreadChecker where needed.
This cl is in preparation for https://codereview.webrtc.org/2060403002/ Add task queue to Call.
In the coming cl the video_sender, and i420_buffer_pool will be used on a task queue and therefore SequencedTaskChecker is needed instead of a ThreadChecker.

BUG=webrtc:5687
Committed: https://crrev.com/ec7cef854d85be2d9fd5b1a4e2c59a67feb662eb
Cr-Commit-Position: refs/heads/master@{#13474}

[perkj_webrtc, 2016-07-13 10:12:00]

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

[perkj_webrtc, 2016-07-13 10:12:01]

please?

[perkj_webrtc, 2016-07-13 10:12:40]

On 2016/07/13 10:12:01, perkj_webrtc wrote:
> please?

oops- let me check the compile error first.

[tommi, 2016-07-13 10:15:47]

looks like you might need to do some more updates, but assuming those are minor
(maybe only update the test?) lgtm. Just ping me if there's something else you'd
like me to take a look at.

https://codereview.webrtc.org/2149553002/diff/1/webrtc/modules/video_coding/v...
File webrtc/modules/video_coding/video_coding.gypi (right):

https://codereview.webrtc.org/2149553002/diff/1/webrtc/modules/video_coding/v...
webrtc/modules/video_coding/video_coding.gypi:17:
'../base/base.gyp:rtc_task_queue',
is this list in correct order?

[perkj_webrtc, 2016-07-13 10:17:38]

On 2016/07/13 10:15:47, tommi-webrtc wrote:
> looks like you might need to do some more updates, but assuming those are
minor
> (maybe only update the test?) lgtm. Just ping me if there's something else
you'd
> like me to take a look at.
> 
>
https://codereview.webrtc.org/2149553002/diff/1/webrtc/modules/video_coding/v...
> File webrtc/modules/video_coding/video_coding.gypi (right):
> 
>
https://codereview.webrtc.org/2149553002/diff/1/webrtc/modules/video_coding/v...
> webrtc/modules/video_coding/video_coding.gypi:17:
> '../base/base.gyp:rtc_task_queue',
> is this list in correct order?

right- the problem is the bad inclusion I am trying to fix in
https://codereview.webrtc.org/2143123003/ that has been reverted a couple of
times due to up stream projects that does not include what they use..... Got to
fix that upstream and reland that cl first.

[perkj_webrtc, 2016-07-14 08:08:51]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2016-07-14 08:08:59]

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, 2016-07-14 08:32:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 08:32:17]

Description was changed from

==========
Switch to use SequencedTaskChecker instead of ThreadChecker where needed.
This cl is in preparation for https://codereview.webrtc.org/2060403002/ Add task
queue to Call.
In the coming cl the video_sender, and i420_buffer_pool will be used on a task
queue and therefore SequencedTaskChecker is needed instead of a ThreadChecker.

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

to

==========
Switch to use SequencedTaskChecker instead of ThreadChecker where needed.
This cl is in preparation for https://codereview.webrtc.org/2060403002/ Add task
queue to Call.
In the coming cl the video_sender, and i420_buffer_pool will be used on a task
queue and therefore SequencedTaskChecker is needed instead of a ThreadChecker.

BUG=webrtc:5687

Committed: https://crrev.com/ec7cef854d85be2d9fd5b1a4e2c59a67feb662eb
Cr-Commit-Position: refs/heads/master@{#13474}
==========

[commit-bot: I haz the power, 2016-07-14 08:32:19]

Patchset 1 (id:??) landed as
https://crrev.com/ec7cef854d85be2d9fd5b1a4e2c59a67feb662eb
Cr-Commit-Position: refs/heads/master@{#13474}

[perkj_webrtc, 2016-07-14 09:33:09]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2150843003/ by perkj@webrtc.org.

The reason for reverting is: Breaks Chrome FYI

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Linux%20Builder/bui....