Make sure that the UI window created by base::MessagePumpForUI is destoyed on the same thread (Wind…

base/message_pump.h

Index: base/message_pump.h
diff --git a/base/message_pump.h b/base/message_pump.h
index b2e0a425656c6962f7d960d3637e68eab48cab36..407f8f866c1501d8a259a25f39105d0e3d88bbea 100644
--- a/base/message_pump.h
+++ b/base/message_pump.h
@@ -119,6 +119,10 @@ class BASE_EXPORT MessagePump : public RefCountedThreadSafe<MessagePump> {
   // used on the thread that called Run.
   virtual void ScheduleDelayedWork(const TimeTicks& delayed_work_time) = 0;
 
+  // Called directly by MessageLoop::~MessageLoop to clean up resources that

[rvargas (doing something else), 2013/06/12 00:22:22]

nit: it's better to document what the method does instead of who calls it... and
it would also be better if the method is named accordingly (note that this file
does not mention MesssageLoop so far).

[alexeypa (please no reviews), 2013/06/12 18:05:22]

Done.

+  // must be freed on the MessageLoop's thread.
+  virtual void WillDestroyCurrentMessageLoop() {}

[rvargas (doing something else), 2013/06/12 00:22:22]

This class is a pure interface... let's keep all methods pure virtual.

[alexeypa (please no reviews), 2013/06/12 18:05:22]

Done.

+
  protected:
   virtual ~MessagePump();
   friend class RefCountedThreadSafe<MessagePump>;