base::Bind: Remove ScopedRunnableMethodFactory.

remoting/base/scoped_thread_proxy.h

 // Copyright (c) 2011 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 REMOTING_BASE_SCOPED_THREAD_PROXY_H_
 #define REMOTING_BASE_SCOPED_THREAD_PROXY_H_
 
 #include "base/callback_forward.h"
 #include "base/message_loop_proxy.h"
 
 namespace remoting {
 
 // ScopedThreadProxy is proxy for message loops that cancels all
 // pending tasks when it is destroyed. It can be used to post tasks
 // for a non-refcounted object. Must be deleted on the thread it
 // belongs to.
 //
-// The main difference from WeakPtr<> and ScopedRunnableMethodFactory<>

[Sergey Ulanov, 2012/01/03 18:49:00]

This comment is still useful to explain why WeakPtr<> cannot be used, just
remove ScopedRunnableMethodFactory<> from it.

[James Hawkins, 2012/01/03 21:13:54]

Done.

-// is that this class can be used safely to post tasks from different
-// threads.
 // It is similar to WeakHandle<> used in sync: the main difference is
 // that WeakHandle<> creates closures itself, while ScopedThreadProxy
 // accepts base::Closure instances which caller needs to create using
 // base::Bind().
 //
 // TODO(sergeyu): Potentially we could use WeakHandle<> instead of
 // this class. Consider migrating to WeakHandle<> when it is moved to
 // src/base and support for delayed tasks is implemented.
 //
 // Usage:
@@ skipping 33 matching lines @@
   class Core;
 
   scoped_refptr<Core> core_;
 
   DISALLOW_COPY_AND_ASSIGN(ScopedThreadProxy);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_BASE_SCOPED_THREAD_PROXY_H_