Readding UI test I pulled yesterday; hopefully the bots will like it more now...

chrome/browser/vista_frame.cc

 // Copyright (c) 2006-2008 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/browser/vista_frame.h"
 
 #include <windows.h>
 #include <atlbase.h>
 #include <atlapp.h>
 #include <atltheme.h>
@@ skipping 631 matching lines @@
 void VistaFrame::UpdateToolbar(TabContents* contents,
                                bool should_restore_state) {
 }
 
 void VistaFrame::ProfileChanged(Profile* profile) {
 }
 
 void VistaFrame::FocusToolbar() {
 }
 
+bool VistaFrame::IsBookmarkBarVisible() const {
+  if (!bookmark_bar_view_.get())
+    return false;
+
+  if (bookmark_bar_view_->IsNewTabPage() || bookmark_bar_view_->IsAnimating())
+    return true;
+
+  CSize sz;
+  bookmark_bar_view_->GetPreferredSize(&sz);
+  // 1 is the minimum in GetPreferredSize for the bookmark bar.
+  return sz.cy > 1;
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 //
 // Events
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 LRESULT VistaFrame::OnSettingChange(UINT u_msg, WPARAM w_param, LPARAM l_param,
                                     BOOL& handled) {
   // Set this to false, if we actually handle the message, we'll set it to
   // true below.
@@ skipping 942 matching lines @@
 
   // Only do a layout if the current contents is non-null. We assume that if the
   // contents is NULL, we're either being destroyed, or ShowTabContents is going
   // to be invoked with a non-null TabContents again so that there is no need
   // in doing a layout now (and would result in extra work/invalidation on
   // tab switches).
   if (changed && current_tab)
     Layout();
 }