Sprinkle some defensiveness into the UI tests so that they don't explode if t...

chrome/test/automated_ui_tests/automated_ui_test_base.cc

 // Copyright (c) 2009 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.
 
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/view_ids.h"
 #include "chrome/test/automated_ui_tests/automated_ui_test_base.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "chrome/test/automation/tab_proxy.h"
 #include "chrome/test/automation/window_proxy.h"
@@ skipping 235 matching lines @@
   return RunCommand(IDC_NEW_INCOGNITO_WINDOW);
 }
 
 bool AutomatedUITestBase::OpenAndActivateNewBrowserWindow(
     scoped_refptr<BrowserProxy>* previous_browser) {
   if (!automation()->OpenNewBrowserWindow(true /* SW_SHOWNORMAL */)) {
     LogWarningMessage("failed_to_open_new_browser_window");
     return false;
   }
   int num_browser_windows;
-  automation()->GetBrowserWindowCount(&num_browser_windows);
+  if (!automation()->GetBrowserWindowCount(&num_browser_windows)) {
+    LogErrorMessage("failed_to_get_browser_window_count");
+    return false;
+  }
   // Get the most recently opened browser window and activate the tab
   // in order to activate this browser window.
   scoped_refptr<BrowserProxy> browser(
       automation()->GetBrowserWindow(num_browser_windows - 1));
   if (browser.get() == NULL) {
     LogErrorMessage("browser_window_not_found");
     return false;
   }
   bool is_timeout;
   if (!browser->ActivateTabWithTimeout(0, action_max_timeout_ms(),
@@ skipping 90 matching lines @@
     BrowserProxy* browser) {
   bool did_timeout;
   if (!browser->BringToFrontWithTimeout(action_max_timeout_ms(),
                                         &did_timeout)) {
     LogWarningMessage("failed_to_bring_window_to_front");
     return NULL;
   }
 
   return browser->GetWindow();
 }