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

chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc

Index: chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc
diff --git a/chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc b/chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc
index 43fcec8bf5d0813469f1e1603944f5e648b94343..25d512ab6127fdb48d76bf20c2c0bb84d754e658 100644
--- a/chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc
+++ b/chrome/browser/ui/fullscreen_controller_interactive_browsertest.cc
@@ -85,6 +85,7 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
 
   ASSERT_FALSE(IsFullscreenBubbleDisplayed());
 
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);

[jbates, 2012/06/13 22:12:27]

Of course, I'll remove these printfs before commit :) Just trying to find the
cause of a timeout in this test.

[jar (doing other things), 2012/06/14 04:50:29]

You could probably use a DLOG() function, and not have to worry about yanking it
in time.

On 2012/06/13 22:12:27, jbates wrote:

   // Request to lock the mouse and enter fullscreen.
   {
     FullscreenNotificationObserver fullscreen_observer;
@@ -92,20 +93,24 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
         browser(), ui::VKEY_B, false, true, false, false,
         chrome::NOTIFICATION_MOUSE_LOCK_CHANGED,
         content::NotificationService::AllSources()));
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
     fullscreen_observer.Wait();
   }
   ASSERT_TRUE(IsFullscreenPermissionRequested());
   ASSERT_TRUE(IsMouseLockPermissionRequested());
 
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   // Escape, no prompts should remain.
   {
     FullscreenNotificationObserver fullscreen_observer;
     SendEscapeToFullscreenController();
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
     fullscreen_observer.Wait();
   }
   ASSERT_FALSE(IsFullscreenPermissionRequested());
   ASSERT_FALSE(IsMouseLockPermissionRequested());
 
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   // Request to lock the mouse and enter fullscreen.
   {
     FullscreenNotificationObserver fullscreen_observer;
@@ -113,11 +118,15 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
         browser(), ui::VKEY_B, false, true, false, false,
         chrome::NOTIFICATION_MOUSE_LOCK_CHANGED,
         content::NotificationService::AllSources()));
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
     fullscreen_observer.Wait();
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   }
   ASSERT_TRUE(IsFullscreenPermissionRequested());
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   ASSERT_TRUE(IsMouseLockPermissionRequested());
 
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   // Accept both, confirm mouse lock and fullscreen and no prompts.
   AcceptCurrentFullscreenOrMouseLockRequest();
   ASSERT_TRUE(IsMouseLocked());
@@ -125,11 +134,15 @@ IN_PROC_BROWSER_TEST_F(FullscreenControllerInteractiveTest,
   ASSERT_FALSE(IsFullscreenPermissionRequested());
   ASSERT_FALSE(IsMouseLockPermissionRequested());
 
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
   // Escape, confirm we are out of mouse lock and fullscreen with no prompts.
   {
     FullscreenNotificationObserver fullscreen_observer;
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
     SendEscapeToFullscreenController();
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);
     fullscreen_observer.Wait();
+fprintf(stderr, "global %s:%s:%d\n", __FILE__, __FUNCTION__, (int)__LINE__);

[jar (doing other things), 2012/06/14 04:50:29]

... I see... you hav a LOT of printfs..... probably should have a final review
when you yank these (before commit).

   }
   ASSERT_FALSE(IsMouseLocked());
   ASSERT_FALSE(IsFullscreenForTabOrPending());