Alternate NTP: Don't hide bookmark bar on instant

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index b519b0eb4e61a6085809ac89d5358eb825bfd4ec..31a5e155e64b8b047b785de01204fe04927e5209 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1260,7 +1260,7 @@ void Browser::MaybeUpdateBookmarkBarStateForInstantPreview(
   // bookmark bar.
   if (mode.is_search_suggestions() &&
       bookmark_bar_state_ == BookmarkBar::SHOW) {
-    UpdateBookmarkBarState(BOOKMARK_BAR_STATE_CHANGE_INSTANT_PREVIEW_STATE);
+    UpdateBookmarkBarState(BOOKMARK_BAR_STATE_CHANGE_TAB_STATE);
   }
 }
 
@@ -2238,11 +2238,15 @@ void Browser::UpdateBookmarkBarState(BookmarkBarStateChangeReason reason) {
 
   // Don't allow the bookmark bar to be shown in suggestions mode or
   // for instant extended api, non-NTP modes when it's detached.
-  if (search_model_->mode().is_search_suggestions() ||
-      (chrome::search::IsInstantExtendedAPIEnabled(profile_) &&
-       !search_model_->mode().is_ntp() && state == BookmarkBar::DETACHED)) {
-    state = BookmarkBar::HIDDEN;
+  // TODO(sail): Stop hidding the bookmark bar on other platforms.

[kuan, 2013/01/18 14:22:11]

since i'm implementing the spacer approach w/out hiding the bmb, u can remove
this todo.

[kuan, 2013/01/18 14:23:16]

correction: WITH hiding the bmb.

[sail, 2013/01/18 16:28:52]

Done.

+#if !defined(OS_MACOSX)
+  if (chrome::search::IsInstantExtendedAPIEnabled(profile_)) {
+    if (search_model_->mode().is_search_suggestions() ||
+        (!search_model_->mode().is_ntp() && state == BookmarkBar::DETACHED)) {
+      state = BookmarkBar::HIDDEN;
+    }
   }
+#endif
 
   if (state == bookmark_bar_state_)
     return;
@@ -2259,15 +2263,20 @@ void Browser::UpdateBookmarkBarState(BookmarkBarStateChangeReason reason) {
     return;
   }
 
+  bool shouldAnimate = reason == BOOKMARK_BAR_STATE_CHANGE_PREF_CHANGE;
+
   // Don't animate if mode is |NTP| because the bookmark is attached at top when
   // pref is on and detached at bottom when off.
-  BookmarkBar::AnimateChangeType animate_type =
-      ((reason == BOOKMARK_BAR_STATE_CHANGE_PREF_CHANGE &&
-        !search_model_->mode().is_ntp()) ||
-       reason == BOOKMARK_BAR_STATE_CHANGE_INSTANT_PREVIEW_STATE) ?
+  // TODO(sail) Remove this once other platforms have removed the bottom
+  // bookmark bar.
+#if !defined(OS_MACOSX)
+  if (chrome::search::IsInstantExtendedAPIEnabled(profile_))
+    shouldAnimate = shouldAnimate && !search_model_->mode().is_ntp();
+#endif

[kuan, 2013/01/18 14:22:11]

my cl to remove bottom bmb just landed at r177659, so u can remove this chunk. 

[sail, 2013/01/18 16:28:52]

Done.

+
+  window_->BookmarkBarStateChanged(shouldAnimate ?
       BookmarkBar::ANIMATE_STATE_CHANGE :
-      BookmarkBar::DONT_ANIMATE_STATE_CHANGE;
-  window_->BookmarkBarStateChanged(animate_type);
+      BookmarkBar::DONT_ANIMATE_STATE_CHANGE);
 }
 
 bool Browser::ShouldHideUIForFullscreen() const {