BookmarkManagerTest now passes without this hack.

chrome/browser/gtk/bookmark_manager_browsertest.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 "base/keyboard_codes.h"
 #include "chrome/browser/automation/ui_controls.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_model_observer.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_window.h"
@@ skipping 63 matching lines @@
 // There was once a regression where we crashed when launching the bookmark
 // manager, and another regression where we crashed when calling
 // RecursiveFind(). This test aims to check for these simple crashes.
 IN_PROC_BROWSER_TEST_F(BookmarkManagerTest, Crash) {
   Init();
 
   // Make sure RecursiveFind() is run.
   manager_->SelectInTree(model_->GetBookmarkBarNode(), true);
 
   CleanUp();
-#if defined(TOOLKIT_VIEWS)
-  // The test fails on toolkit_views build without the following
-  // cleanup code.  This is because the bookmark bar view refuses to
-  // be destroyed by a bookmark model when the BrowserProcessImpl
-  // destroys the bookmark bar model.
-  // TODO(oshima): http://crbug/28046.
-  browser()->window()->Close();
-  MessageLoopForUI::current()->RunAllPending();
-#endif
 }