Fix up various browser tests not to use WaitForNavigation.

chrome/test/base/ui_test_utils.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_TEST_BASE_UI_TEST_UTILS_H_
 #define CHROME_TEST_BASE_UI_TEST_UTILS_H_
 #pragma once
 
 #include <map>
 #include <queue>
@@ skipping 211 matching lines @@
                bool case_sensitive,
                int* ordinal);
 
 // Returns true if the View is focused.
 bool IsViewFocused(const Browser* browser, ViewID vid);
 
 // Simulates a mouse click on a View in the browser.
 void ClickOnView(const Browser* browser, ViewID vid);
 
 // Blocks until a notification for given |type| is received.
+// TODO(gbillock): remove this race hazard.
+// Use WindowedNotificationObserver instead.
 void WaitForNotification(int type);
 
 // Blocks until a notification for given |type| from the specified |source|
 // is received.
+// TODO(gbillock): remove this race hazard.
+// Use WindowedNotificationObserver instead.
 void WaitForNotificationFrom(int type,
                              const NotificationSource& source);
 
 // Register |observer| for the given |type| and |source| and run
 // the message loop until the observer posts a quit task.
 void RegisterAndWait(NotificationObserver* observer,
                      int type,
                      const NotificationSource& source);
 
 // Blocks until |model| finishes loading.
@@ skipping 151 matching lines @@
 //   WindowedNotificationObserver signal(...)
 //   PerformAction()
 //   signal.Wait()
 class WindowedNotificationObserver : public NotificationObserver {
  public:
   // Register to listen for notifications of the given type from either a
   // specific source, or from all sources if |source| is
   // NotificationService::AllSources().
   WindowedNotificationObserver(int notification_type,
                                const NotificationSource& source);
+

[Paweł Hajdan Jr., 2011/08/04 17:56:48]

nit: I'd prefer not to add an empty line here.

[Greg Billock, 2011/08/11 22:57:45]

Done.

   virtual ~WindowedNotificationObserver();
 
   // Wait until the specified notification occurs.  If the notification was
   // emitted between the construction of this object and this call then it
   // returns immediately.
   void Wait();
 
   // WaitFor waits until the given notification type is received from the
   // given object. If the notification was emitted between the construction of
   // this object and this call then it returns immediately.
@@ skipping 174 matching lines @@
 
 // Takes a snapshot of the entire page, according to the width and height
 // properties of the DOM's document. Returns true on success. DOMAutomation
 // must be enabled.
 bool TakeEntirePageSnapshot(RenderViewHost* rvh,
                             SkBitmap* bitmap) WARN_UNUSED_RESULT;
 
 }  // namespace ui_test_utils
 
 #endif  // CHROME_TEST_BASE_UI_TEST_UTILS_H_