Revert 113183 - Revert 113177 - process all ui events before posting closure because menu depends...

chrome/test/base/view_event_test_base.cc

Index: chrome/test/base/view_event_test_base.cc
===================================================================
--- chrome/test/base/view_event_test_base.cc	(revision 113187)
+++ chrome/test/base/view_event_test_base.cc	(working copy)
@@ -70,9 +70,11 @@
   PostMessage(window_->GetNativeWindow(), WM_USER, 0, 0);
 #endif
 
-  // If we're in a nested message loop, as is the case with menus, we need
-  // to quit twice. The second quit does that for us.
-  MessageLoop::current()->PostTask(FROM_HERE, MessageLoop::QuitClosure());
+  // If we're in a nested message loop, as is the case with menus, we
+  // need to quit twice. The second quit does that for us. Finish all
+  // pending UI events before posting closure because events it may be
+  // executed before UI events are executed.
+  ui_controls::RunClosureAfterAllPendingUIEvents(MessageLoop::QuitClosure());
 }
 
 void ViewEventTestBase::SetUp() {