Add ThreadChecker to the TimestampAligner class.

Add ThreadChecker to the TimestampAligner class.

BUG=

Committed: https://crrev.com/76f91cd08fa4125693450dcb59e2943d8d7110fc
Cr-Commit-Position: refs/heads/master@{#13882}

[nisse-webrtc, 2016-08-23 11:57:02]

nisse@webrtc.org changed reviewers:
+ magjed@webrtc.org, perkj@webrtc.org

[nisse-webrtc, 2016-08-23 11:57:03]

Do you think this is the way to do it?

Personally, I think I'd prefer if the TimestampAligner could be left totally
unaware of threading (and leave it to the caller to use locks or thread jumps or
whatever is most convenient).

And instead add one more thread checker to VideoCapturer, checked only by
AdaptFrame.

[magjed_webrtc, 2016-08-23 12:53:24]

On 2016/08/23 11:57:03, nisse-webrtc wrote:
> Do you think this is the way to do it?
> 
> Personally, I think I'd prefer if the TimestampAligner could be left totally
> unaware of threading (and leave it to the caller to use locks or thread jumps
or
> whatever is most convenient).
> 
> And instead add one more thread checker to VideoCapturer, checked only by
> AdaptFrame.

Hmm, yeah maybe you are right and it's too restrictive to enforce single thread
use. Do we have any helper class for checking that only one thread enters at a
time? 

lgtm. I'm ok if you want to land this to get safety now, and if we need to
generalize it in the future we take care of it then.

[perkj_webrtc, 2016-08-23 18:15:50]

lgtm

[nisse-webrtc, 2016-08-24 06:52:18]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2016-08-24 06:52:22]

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-08-24 08:58:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 08:58:55]

Description was changed from

==========
Add ThreadChecker to the TimestampAligner class.

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

to

==========
Add ThreadChecker to the TimestampAligner class.

BUG=

Committed: https://crrev.com/76f91cd08fa4125693450dcb59e2943d8d7110fc
Cr-Commit-Position: refs/heads/master@{#13882}
==========

[commit-bot: I haz the power, 2016-08-24 08:58:59]

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