Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(562)

Side by Side Diff: webrtc/rtc_base/task_queue_gcd.cc

Issue 3004873002: Delete static method TaskQueue::IsCurrent. (Closed)
Patch Set: Created 3 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « webrtc/rtc_base/task_queue.h ('k') | webrtc/rtc_base/task_queue_libevent.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2016 The WebRTC Project Authors. All rights reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 // a race between checking the context and using it from a task. 138 // a race between checking the context and using it from a task.
139 dispatch_sync_f(queue_, context_, &QueueContext::SetNotActive); 139 dispatch_sync_f(queue_, context_, &QueueContext::SetNotActive);
140 dispatch_release(queue_); 140 dispatch_release(queue_);
141 } 141 }
142 142
143 // static 143 // static
144 TaskQueue* TaskQueue::Current() { 144 TaskQueue* TaskQueue::Current() {
145 return static_cast<TaskQueue*>(pthread_getspecific(GetQueuePtrTls())); 145 return static_cast<TaskQueue*>(pthread_getspecific(GetQueuePtrTls()));
146 } 146 }
147 147
148 // static
149 bool TaskQueue::IsCurrent(const char* queue_name) {
150 TaskQueue* current = Current();
151 return current &&
152 strcmp(queue_name, dispatch_queue_get_label(current->queue_)) == 0;
153 }
154
155 bool TaskQueue::IsCurrent() const { 148 bool TaskQueue::IsCurrent() const {
156 RTC_DCHECK(queue_); 149 RTC_DCHECK(queue_);
157 return this == Current(); 150 return this == Current();
158 } 151 }
159 152
160 void TaskQueue::PostTask(std::unique_ptr<QueuedTask> task) { 153 void TaskQueue::PostTask(std::unique_ptr<QueuedTask> task) {
161 auto* context = new TaskContext(context_, std::move(task)); 154 auto* context = new TaskContext(context_, std::move(task));
162 dispatch_async_f(queue_, context, &TaskContext::RunTask); 155 dispatch_async_f(queue_, context, &TaskContext::RunTask);
163 } 156 }
164 157
(...skipping 12 matching lines...) Expand all
177 context_, std::move(task), reply_queue->context_, std::move(reply)); 170 context_, std::move(task), reply_queue->context_, std::move(reply));
178 dispatch_async_f(queue_, context, &PostTaskAndReplyContext::RunTask); 171 dispatch_async_f(queue_, context, &PostTaskAndReplyContext::RunTask);
179 } 172 }
180 173
181 void TaskQueue::PostTaskAndReply(std::unique_ptr<QueuedTask> task, 174 void TaskQueue::PostTaskAndReply(std::unique_ptr<QueuedTask> task,
182 std::unique_ptr<QueuedTask> reply) { 175 std::unique_ptr<QueuedTask> reply) {
183 return PostTaskAndReply(std::move(task), std::move(reply), Current()); 176 return PostTaskAndReply(std::move(task), std::move(reply), Current());
184 } 177 }
185 178
186 } // namespace rtc 179 } // namespace rtc
OLDNEW
« no previous file with comments | « webrtc/rtc_base/task_queue.h ('k') | webrtc/rtc_base/task_queue_libevent.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698