base: Mark MessageLoop::Post*Task API deprecated

base/message_loop/message_loop.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_MESSAGE_LOOP_MESSAGE_LOOP_H_
 #define BASE_MESSAGE_LOOP_MESSAGE_LOOP_H_
 
 #include <queue>
 #include <string>
 
@@ skipping 137 matching lines @@
   };
 
   // Add a DestructionObserver, which will start receiving notifications
   // immediately.
   void AddDestructionObserver(DestructionObserver* destruction_observer);
 
   // Remove a DestructionObserver.  It is safe to call this method while a
   // DestructionObserver is receiving a notification callback.
   void RemoveDestructionObserver(DestructionObserver* destruction_observer);
 
+  // NOTE: Deprecated; prefer task_runner() and the TaskRunner interfaces.

[Wez, 2015/03/20 04:13:01]

Is there a cleanup bug filed to update call-sites and eventually remove these?

[Sami, 2015/03/20 17:16:56]

I was going to do as part of the linked bug. I've got a mostly working clang
tool for it: https://codereview.chromium.org/1010073002/

[danakj, 2015/03/20 17:19:12]

Linking to bugs in TODO is a good practice.

+  //
   // The "PostTask" family of methods call the task's Run method asynchronously
   // from within a message loop at some point in the future.
   //
   // With the PostTask variant, tasks are invoked in FIFO order, inter-mixed
   // with normal UI or IO event processing.  With the PostDelayedTask variant,
   // tasks are called after at least approximately 'delay_ms' have elapsed.
   //
   // The NonNestable variants work similarly except that they promise never to
   // dispatch the task from a nested invocation of MessageLoop::Run.  Instead,
   // such tasks get deferred until the top-most MessageLoop::Run is executing.
@@ skipping 481 matching lines @@
 
 // Do not add any member variables to MessageLoopForIO!  This is important b/c
 // MessageLoopForIO is often allocated via MessageLoop(TYPE_IO).  Any extra
 // data that you need should be stored on the MessageLoop's pump_ instance.
 COMPILE_ASSERT(sizeof(MessageLoop) == sizeof(MessageLoopForIO),
                MessageLoopForIO_should_not_have_extra_member_variables);
 
 }  // namespace base
 
 #endif  // BASE_MESSAGE_LOOP_MESSAGE_LOOP_H_