BookmarkManagerTest now passes without this hack.

chrome/browser/gtk/bookmark_manager_browsertest.cc

Index: chrome/browser/gtk/bookmark_manager_browsertest.cc
diff --git a/chrome/browser/gtk/bookmark_manager_browsertest.cc b/chrome/browser/gtk/bookmark_manager_browsertest.cc
index 1d8b8b00167ba5a4181ea8857957db5775480851..6cb32894e76f401a98a6a36e296039bd868dd721 100644
--- a/chrome/browser/gtk/bookmark_manager_browsertest.cc
+++ b/chrome/browser/gtk/bookmark_manager_browsertest.cc
@@ -81,13 +81,4 @@ IN_PROC_BROWSER_TEST_F(BookmarkManagerTest, Crash) {
   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
 }