Delete static method TaskQueue::IsCurrent.

Delete static method TaskQueue::IsCurrent.

The static method takes a queue name as argument. Used only in one
place, a DCHECK in IncomingVideoStream, which is converted to use
the non-static version of IsCurrent.

BUG=webrtc:8166
Review-Url: https://codereview.webrtc.org/3004873002
Cr-Commit-Position: refs/heads/master@{#19657}
Committed: https://chromium.googlesource.com/external/webrtc/+/2c7b7a633f6a6aef4467cc3b35a056a4dcc9d619

[nisse-webrtc, 2017-08-30 08:38:10]

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

[nisse-webrtc, 2017-08-30 08:38:11]

PTAL. Any reason to keep the method

  static bool IsCurrent(const char* queue_name);

?

The more implementations of TaskQueue we get, the more important it is to keep
API size down.

Not sure how to delete it without temporarily breaking downstream overrides, any
advice?

[tommi, 2017-08-30 09:49:56]

lgtm fwiw

[magjed_webrtc, 2017-08-30 11:29:30]

webrtc/common_video/incoming_video_stream.cc lgtm

[perkj_webrtc, 2017-09-04 08:37:54]

lgtm. Will you also update Chromes header file? It use a copy of the header file
at the moment.

[nisse-webrtc, 2017-09-04 08:44:17]

On 2017/09/04 08:37:54, perkj_webrtc wrote:
> lgtm. Will you also update Chromes header file? It use a copy of the header
file
> at the moment.

I can do that. But I should land this cl first, right?

[nisse-webrtc, 2017-09-04 09:43:19]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2017-09-04 09:43:32]

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-09-04 12:18:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1504518199139010, "parent_rev":
"334f9e6a8d67f646b1a60a1683046224c351c5a4", "commit_rev":
"2c7b7a633f6a6aef4467cc3b35a056a4dcc9d619"}

[commit-bot: I haz the power, 2017-09-04 12:18:26]

Description was changed from

==========
Delete static method TaskQueue::IsCurrent.

The static method takes a queue name as argument. Used only in one
place, a DCHECK in IncomingVideoStream, which is converted to use
the non-static version of IsCurrent.

BUG=webrtc:8166
==========

to

==========
Delete static method TaskQueue::IsCurrent.

The static method takes a queue name as argument. Used only in one
place, a DCHECK in IncomingVideoStream, which is converted to use
the non-static version of IsCurrent.

BUG=webrtc:8166

Review-Url: https://codereview.webrtc.org/3004873002
Cr-Commit-Position: refs/heads/master@{#19657}
Committed:
https://chromium.googlesource.com/external/webrtc/+/2c7b7a633f6a6aef4467cc3b3...
==========

[commit-bot: I haz the power, 2017-09-04 12:18:28]

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