Add base::RunLoop and update ui_test_utils to use it to reduce flakiness

chrome/test/base/in_process_browser_test.cc

Index: chrome/test/base/in_process_browser_test.cc
diff --git a/chrome/test/base/in_process_browser_test.cc b/chrome/test/base/in_process_browser_test.cc
index a65d00cbe3f998e07e2bcc8c73ca59141bf3ab76..0719a0cbafcbcae428178021f59625b34f023b49 100644
--- a/chrome/test/base/in_process_browser_test.cc
+++ b/chrome/test/base/in_process_browser_test.cc
@@ -373,6 +373,10 @@ void InProcessBrowserTest::RunTestOnMainThreadLoop() {
   autorelease_pool_->Recycle();
 #endif
 
+  // Sometimes tests leave Quit tasks in the MessageLoop (for shame), so let's

[Paweł Hajdan Jr., 2012/06/27 07:33:28]

nit: Please add TODO(phajdan.jr) to maybe detect those cases and make such bad
tests fail. Feel free to make that TODO(jbates).

[jbates, 2012/06/28 20:46:11]

Done.

+  // run all pending messages here to avoid preempting the QuitBrowsers tasks.
+  ui_test_utils::RunAllPendingInMessageLoop();
+
   QuitBrowsers();
   CHECK(BrowserList::empty());
 }