Commit 6cb615b7 authored by Gabriel Charette's avatar Gabriel Charette Committed by Commit Bot

Remove unnecessary TODO in browser_thread.h

As discussed @ https://crbug.com/878356#c39

R=thestig@chromium.org

Bug: 878356
Change-Id: I7f9df5e84384c4f0c61a4ac02624fec54e6d9a27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1822419Reviewed-by: default avatarLei Zhang <thestig@chromium.org>
Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
Reviewed-by: default avatarRohit Rao <rohitrao@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Auto-Submit: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699829}
parent f6a5f5b8
......@@ -69,8 +69,13 @@ class CONTENT_EXPORT BrowserThread {
// NOTE: Task posting APIs have moved to post_task.h. See
// browser_task_traits.h.
// TODO(crbug.com/878356): Consider replacing callsites of this with
// base::CreateTaskRunner({id})->DeleteSoon(..).
// Delete/ReleaseSoon() helpers allow future deletion of an owned object on
// its associated thread. If you already have a task runner bound to a
// BrowserThread you should use its SequencedTaskRunner::DeleteSoon() member
// method. If you don't, the helpers below avoid having to do
// base::CreateSingleThreadTaskRunner({BrowserThread::ID})->DeleteSoon(...)
// which is equivalent.
template <class T>
static bool DeleteSoon(ID identifier,
const base::Location& from_here,
......
......@@ -67,8 +67,13 @@ class WebThread {
// NOTE: Task posting APIs have moved to post_task.h. See web_task_traits.h.
// TODO(crbug.com/878356): Consider replacing callsites of this with
// base::CreateTaskRunner({id})->DeleteSoon(..).
// Delete/ReleaseSoon() helpers allow future deletion of an owned object on
// its associated thread. If you already have a task runner bound to a
// WebThread you should use its SequencedTaskRunner::DeleteSoon() member
// method. If you don't, the helpers below avoid having to do
// base::CreateSingleThreadTaskRunner({WebThread::ID})->DeleteSoon(...) which
// is equivalent.
template <class T>
static bool DeleteSoon(ID identifier,
const base::Location& from_here,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment