alternate ntp: implement Show/HideBars API to reduce jank when showing/hiding bars

chrome/browser/ui/search/search_model.cc

 // Copyright (c) 2012 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/ui/search/search_model.h"
 
 #include "chrome/browser/instant/search.h"
 #include "chrome/browser/ui/search/search_model_observer.h"
 
 namespace chrome {
 namespace search {
 
-SearchModel::SearchModel() {
+SearchModel::SearchModel() : top_bars_visible_(true) {
 }
 
 SearchModel::~SearchModel() {
 }
 
 void SearchModel::SetMode(const Mode& new_mode) {
   DCHECK(IsInstantExtendedAPIEnabled())
       << "Please do not try to set the SearchModel mode without first "
       << "checking if Search is enabled.";
 
   if (mode_ == new_mode)
     return;
 
   const Mode old_mode = mode_;
   mode_ = new_mode;
 
+  // For |SEARCH_SUGGESTIONS| and |SEARCH_RESULTS| modes, SearchBox API will
+  // determine visibility of top bars via SetTopBarsVisible(); for other modes,
+  // top bars are always visible, if available.
+  if (!mode_.is_search())
+    top_bars_visible_ = true;
+
   FOR_EACH_OBSERVER(SearchModelObserver, observers_,
-                    ModeChanged(old_mode, mode_));
+                    ModelChanged(old_mode, mode_, top_bars_visible_));
+}
+
+void SearchModel::SetTopBarsVisible(bool visible) {
+  DCHECK(IsInstantExtendedAPIEnabled())
+      << "Please do not try to set the SearchModel mode without first "
+      << "checking if Search is enabled.";
+
+  top_bars_visible_ = visible;

[dhollowa, 2013/03/13 17:21:55]

Do an early return if there's no change to the visibility.

[kuan, 2013/03/13 22:21:07]

Done.

+

[dhollowa, 2013/03/13 17:21:55]

Save off old_state_  here so the receiver can determine what changed.  i.e.:

  FOE(..., ModelChanged(old_state, state_));

[kuan, 2013/03/13 22:21:07]

Done.

+  FOR_EACH_OBSERVER(SearchModelObserver, observers_,
+                    ModelChanged(mode_, mode_, top_bars_visible_));
 }
 
 void SearchModel::AddObserver(SearchModelObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void SearchModel::RemoveObserver(SearchModelObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
 } //  namespace search
 } //  namespace chrome