Addition testing for panels drag and drop.

chrome/browser/ui/panels/panel_manager.h

Index: chrome/browser/ui/panels/panel_manager.h
diff --git a/chrome/browser/ui/panels/panel_manager.h b/chrome/browser/ui/panels/panel_manager.h
index 1ece64e120af31f8cac463b18e707199e876d5f6..22bfa009dc03489f870609ac4d047b16175211a3 100644
--- a/chrome/browser/ui/panels/panel_manager.h
+++ b/chrome/browser/ui/panels/panel_manager.h
@@ -84,6 +84,9 @@ class PanelManager {
   void DragLeft();
   void DragRight();
 
+  // Horizontal spacing between panels.  Used for unit testing.
+  static int horizontal_spacing();

[jianli, 2011/08/24 20:23:24]

Can you make it public and add UNIT_TEST guard for it? You can move the
definition of constant to this header file.

Also, do not make it static method. Use
PanelManager::GetInstance()->horizontal_spacing() to access it in the calling
code, for consistency reason.

[prasadt, 2011/08/24 21:29:17]

I'd rather not do this change.  If we move one definition, we have to move
others too to keep it consistent but no one else needs them, so they really
belong in the cc file.

Also it makes more sense for this function to be a static because its just
returning a constant.

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

We're trying to avoid accessing private methods and members directly from the
testing code.

+
   Panels panels_;
 
   // Stores the panels that are pending to remove. We want to delay the removal