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..d5295339373e127d7fff3ec45322aa9a382c8be1 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;
 
+  // Stops the pump and frees any resources that must to be freed on the thread
+  // that called Run(). This is the last method called on that thread.

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

I know what you want to say, but I don't know how to read the last sentence. It
should say "this must be the last", but that is not correct. "should be" is
closer, but also not true (as in nothing really happens if, say, ScheduleWork()
is called on that thread after Stop... it just doesn't do anything.

How about "Stops the pump and frees any resources that must to be freed on the
thread that called Run(). This method should be called on the same thread that
called Run()"... I still don't like how that reads, but it is explicit about
thread expectations, but lacks the detail about the pump doing nothing after
this point; maybe that's clear enough?

[alexeypa (please no reviews), 2013/06/13 00:13:40]

Done.

+  virtual void Stop() = 0;
+
  protected:
   virtual ~MessagePump();
   friend class RefCountedThreadSafe<MessagePump>;