Addition testing for panels drag and drop.

chrome/browser/ui/panels/panel_manager.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_MANAGER_H_
 #define CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_
 #pragma once
 
 #include <vector>
 #include "base/basictypes.h"
@@ skipping 37 matching lines @@
   int num_panels() const { return panels_.size(); }
   bool is_dragging_panel() const;
 
  private:
   friend class PanelBrowserTest;
 
   typedef std::vector<Panel*> Panels;
 
   PanelManager();
 
+#if UNIT_TEST
+  Panels panels() { return panels_; }

[jianli, 2011/08/24 22:04:03]

Please define it as:
  const Panels& panels() const { return panels_; }

[prasadt, 2011/08/24 23:04:30]

Done.

+#endif
+
   // Applies the new work area. This is called by OnDisplayChanged and the test
   // code.
   void SetWorkArea(const gfx::Rect& work_area);
 
   // Handles all the panels that're delayed to be removed.
   void DelayedRemove();
 
   // Does the remove. Called from Remove and DelayedRemove.
   void DoRemove(Panel* panel);
 
   // Rearranges the positions of the panels starting from the given iterator.
   // This is called when the display space has been changed, i.e. working
   // area being changed or a panel being closed.
   void Rearrange(Panels::iterator iter_to_start);
 
   // Computes the bounds for next panel.
   // |allow_size_change| is used to indicate if the panel size can be changed to
   // fall within the size constraint, e.g., when the panel is created.
   // Returns true if computed bounds are within the displayable area.
   bool ComputeBoundsForNextPanel(gfx::Rect* bounds, bool allow_size_change);
 
   // Finds one panel to close so that we may have space for the new panel
   // created by |extension|.
   void FindAndClosePanelOnOverflow(const Extension* extension);
 
   // Help functions to drag the given panel.
   void DragLeft();
   void DragRight();
 
+  // Horizontal spacing between panels.  Used for unit testing.
+  static int horizontal_spacing();
+
   Panels panels_;
 
   // Stores the panels that are pending to remove. We want to delay the removal
   // when we're in the process of the dragging.
   Panels panels_pending_to_remove_;
 
   // Current work area used in computing the panel bounds.
   gfx::Rect work_area_;
 
   // Used in computing the bounds of the next panel.
@@ skipping 12 matching lines @@
 
   // Bounds of the panel to drag. It is first set to the original bounds when
   // the dragging happens. Then it is updated to the position that will be set
   // to when the dragging ends.
   gfx::Rect dragging_panel_bounds_;
 
   DISALLOW_COPY_AND_ASSIGN(PanelManager);
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_PANEL_MANAGER_H_