Uncomment commented-out sequence-checks in call.cc

Uncomment commented-out sequence-checks in call.cc

Two of the three commented-out checks should have been fixed by https://codereview.webrtc.org/2998923002/. The last will be fixed in a separate CL.

BUG=webrtc:8116
Review-Url: https://codereview.webrtc.org/2999253002
Cr-Commit-Position: refs/heads/master@{#19511}
Committed: https://chromium.googlesource.com/external/webrtc/+/d1dd2f79e5f290ef0f21f5c050aa0111732a0f71

[eladalon, 2017-08-18 14:37:57]

eladalon@webrtc.org changed reviewers:
+ solenberg@webrtc.org, stefan@webrtc.org

[eladalon, 2017-08-18 14:37:58]

PTAL

https://codereview.webrtc.org/2999253002/diff/1/webrtc/call/call.cc
File webrtc/call/call.cc (right):

https://codereview.webrtc.org/2999253002/diff/1/webrtc/call/call.cc#newcode907
webrtc/call/call.cc:907: // TODO(solenberg): Some test cases in EndToEndTest use
this from a different
We can take care of this separately, as it requires another change, and I want
to also make sure the TODO is still relevant, meaning that no calls to
GetStats() from any other threads have been added.

[stefan-webrtc, 2017-08-20 11:25:51]

LGTM, but there might be some problems with linux_internal

[eladalon, 2017-08-25 08:52:59]

I don't remember if those problems were due to problems with importing WebRTC,
or due to some issues with the underlying CL. Either way, they're now fixed. :-)

[eladalon, 2017-08-25 08:53:04]

The CQ bit was checked by eladalon@webrtc.org

[eladalon, 2017-08-25 08:53:05]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2999253002/#ps40001 (title:
"Rebase")

[commit-bot: I haz the power, 2017-08-25 08:53:07]

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-08-25 09:55:58]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1503651184489700,
"parent_rev": "27e812e36927e15401259e6a1a2cd01b9fdd4f79", "commit_rev":
"d1dd2f79e5f290ef0f21f5c050aa0111732a0f71"}

[commit-bot: I haz the power, 2017-08-25 09:56:06]

Description was changed from

==========
Uncomment commented-out sequence-checks in call.cc

Two of the three commented-out checks should have been fixed by
https://codereview.webrtc.org/2998923002/. The last will be fixed in a separate
CL.

BUG=webrtc:8116
==========

to

==========
Uncomment commented-out sequence-checks in call.cc

Two of the three commented-out checks should have been fixed by
https://codereview.webrtc.org/2998923002/. The last will be fixed in a separate
CL.

BUG=webrtc:8116

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

[commit-bot: I haz the power, 2017-08-25 09:56:07]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/d1dd2f79e5f290ef0f21f5c05...