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 33 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 {

[jennb, 2011/12/08 23:52:00]

Use a TestPanelMouseWatcher instance and simulate mouse move instead of adding
API. See other tests in panel_browsertest.cc for examples.

[jianli, 2011/12/09 22:39:07]

I've already used TestPanelMouseWatcher that. But unfortunately it is not enough
to cover the cases I want to test. I want to check if the expanded overflow area
is still shown when the mouse moves to any other point within this area.

+    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;
+
  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_