Rename TaskAnnotator::DidQueueTask to better represent semantics

Per the move-only nature of PendingTask (OnceClosure), DidQueueTask()
already had to be invoked *before* queueing the task.

Furthermore, it did modify metadata of the PendingTask and as such
calling it after moving the task would have usually been wrong.
(me aculpa : I'm the one that made task_backtrace mutable to enable this
const& modification..!)

This confused me when writing
https://chromium-review.googlesource.com/c/chromium/src/+/1127262
and the upcoming task source observer interface will need a notion of
WillQueueTask() vs DidQueueTask().

This change was extracted from it since this touches a lot of files.

TBR=derat@chromium.org (alarm_timer_chromeos.cc side-effect)

Bug: 860252
Change-Id: I41328fa25a929f602672e3c11176e62906a19d95
Reviewed-on: https://chromium-review.googlesource.com/1127263
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Fran├žois Doray <fdoray@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#573014}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 4036f8289df032ccc7bbf43e1a295d2a6db613ec
1 file changed