Add test cases for panel overflow handling.

chrome/browser/ui/panels/panel_overflow_strip.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_UI_PANELS_PANEL_OVERFLOW_STRIP_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_OVERFLOW_STRIP_H_
 #pragma once
 
 #include <vector>
 #include "chrome/browser/ui/panels/panel.h"
@@ skipping 32 matching lines @@
 
   // Refreshes the layouts for all panels to reflect any possible changes.
   void Refresh();
 
   void OnFullScreenModeChanged(bool is_full_screen);
 
   int num_panels() const { return static_cast<int>(panels_.size()); }
   Panel* first_panel() const {
     return panels_.empty() ? NULL : panels_.front();
   }
+  const Panels& panels() const { return panels_; }
 
 #ifdef UNIT_TEST
-  const Panels& panels() const { return panels_; }
+  void set_max_visible_panels(int max_visible_panels) {
+    max_visible_panels_ = max_visible_panels;
+  }
+
+  bool TestShouldShowOverflowTitles(const gfx::Point& mouse_position) const {
+    return ShouldShowOverflowTitles(mouse_position);
+  }
 #endif
 
+  // The width of the overflow area that is expanded to show more info, i.e.
+  // titles, when the mouse hovers over the area.
+  static const int kOverflowAreaHoverWidth = 200;

[jennb, 2011/12/10 00:42:46]

Don't need this here if overflow strip has a notion of
current_width/height/bounds with a getter for unit_test. Might also make
overflow strip logic more straightforward, too, as discussed offline.

[jianli, 2011/12/13 23:23:40]

Done.

+
  private:
   // Overridden from PanelMouseWatcherObserver:
   virtual void OnMouseMove(const gfx::Point& mouse_position) OVERRIDE;
 
   // Overridden from AnimationDelegate:
+  virtual void AnimationEnded(const ui::Animation* animation) OVERRIDE;
   virtual void AnimationProgressed(const ui::Animation* animation) OVERRIDE;
 
   void DoRefresh(size_t start_index, size_t end_index);
 
   // Computes the layout of the |index| panel to fit into the area.
   // Empty bounds will be returned if this is not possible due to not enough
   // space.
   gfx::Rect ComputeLayout(size_t index,
                           const gfx::Size& iconified_size) const;
 
   // Used to pop up the titles for overflow panels when the mouse hovers over
   // the overflow area.
   bool ShouldShowOverflowTitles(const gfx::Point& mouse_position) const;
   void ShowOverflowTitles(bool show_overflow_titles);
 
   // Weak pointer since PanelManager owns PanelOverflowStrip instance.
   PanelManager* panel_manager_;
 
   // The queue for storing all panels.
   Panels panels_;
 
   // The overflow area where panels are iconified due to insufficient space
  // in the panel strip.
   gfx::Rect display_area_;
 
+  // Maximium number of overflow panels allowed to be shown.
+  int max_visible_panels_;
+
   // For mouse hover-over effect.
   bool are_overflow_titles_shown_;
   scoped_ptr<ui::SlideAnimation> overflow_hover_animator_;
   int overflow_hover_animator_start_width_;
   int overflow_hover_animator_end_width_;
 
   // Invalid panel index.
   static const size_t kInvalidPanelIndex = static_cast<size_t>(-1);
 
   DISALLOW_COPY_AND_ASSIGN(PanelOverflowStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_OVERFLOW_STRIP_H_