base::Bind: Cleanup in automation.

base/message_loop.h

Index: base/message_loop.h
diff --git a/base/message_loop.h b/base/message_loop.h
index 29c736c25aafdaaccbbc7b3e7d0b7554384cb2f4..1cebddc1fa1b9e8d94d9268b107553a1283ece33 100644
--- a/base/message_loop.h
+++ b/base/message_loop.h
@@ -11,6 +11,7 @@
 
 #include "base/base_export.h"
 #include "base/basictypes.h"
+#include "base/bind.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/memory/ref_counted.h"
@@ -245,7 +246,7 @@ class BASE_EXPORT MessageLoop : public base::MessagePump::Delegate {
   // Use QuitTask if you need to Quit another thread's MessageLoop, but note
   // that doing so is fairly dangerous if the target thread makes nested calls
   // to MessageLoop::Run.  The problem being that you won't know which nested
-  // run loop you are quiting, so be careful!
+  // run loop you are quitting, so be careful!
   //
   void Quit();
 
@@ -262,6 +263,11 @@ class BASE_EXPORT MessageLoop : public base::MessagePump::Delegate {
     }
   };
 
+  static base::Closure QuitClosure() {
+    return base::Bind(&MessageLoop::Quit,

[willchan no longer on Chromium, 2011/10/16 02:51:33]

We're trying to keep bind.h out of header files. Can you move this to the .cc
file? I see that this rule has already been broken, but message_loop.h is pretty
far-reaching, so I think it's worthwhile to keep it out of this header too
unless there's a good reason to keep it here (optimizing this function seems
unnecessary).

[James Hawkins, 2011/10/16 22:00:24]

Oops. Done.

+                      base::Unretained(MessageLoop::current()));
+  }
+
   // Returns the type passed to the constructor.
   Type type() const { return type_; }