Add some more EXPECT_TRUE to track down the valgrind issue....

chrome/browser/dom_ui/bookmarks_ui_uitest.cc

 // Copyright (c) 2010 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/test/ui/ui_test.h"
 
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/automation/browser_proxy.h"
 #include "chrome/test/automation/tab_proxy.h"
@@ skipping 15 matching lines @@
 
   scoped_refptr<TabProxy> GetBookmarksUITab() {
     scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
     EXPECT_TRUE(browser.get());
     if (!browser.get())
       return NULL;
     scoped_refptr<TabProxy> tab = browser->GetActiveTab();
     EXPECT_TRUE(tab.get());
     if (!tab.get())
       return NULL;
-    if (!tab->NavigateToURL(GURL(chrome::kChromeUIBookmarksURL)))
+    bool success = tab->NavigateToURL(GURL(chrome::kChromeUIBookmarksURL));
+    EXPECT_TRUE(success);
+    if (!success)
       return NULL;
-    if (!WaitForBookmarksUI(tab))
+    success = WaitForBookmarksUI(tab);
+    EXPECT_TRUE(success);
+    if (!success)
       return NULL;
     return tab;
   }
 
   void AssertIsBookmarksPage(TabProxy* tab) {
     // tab->GetCurrentURL is not up to date.
     GURL url;
     std::wstring out;
     ASSERT_TRUE(tab->ExecuteAndExtractString(L"",
         L"domAutomationController.send(location.protocol)", &out));
@@ skipping 43 matching lines @@
 
   ASSERT_TRUE(WaitForBookmarksUI(tab));
 
   AssertIsBookmarksPage(tab);
 }
 
 TEST_F(BookmarksUITest, BookmarksLoaded) {
   scoped_refptr<TabProxy> tab = GetBookmarksUITab();
   ASSERT_TRUE(tab.get());
 }