Implement drag and drop testing in a platform independent way.

chrome/browser/ui/panels/native_panel.h

Index: chrome/browser/ui/panels/native_panel.h
diff --git a/chrome/browser/ui/panels/native_panel.h b/chrome/browser/ui/panels/native_panel.h
index a67fe19f9589a65c31b1ef45c51f0c849e18b604..310ce0cc19e736e43b31a4330baa5884b5e2ae76 100644
--- a/chrome/browser/ui/panels/native_panel.h
+++ b/chrome/browser/ui/panels/native_panel.h
@@ -11,6 +11,7 @@
 
 class FindBar;
 class NativePanelTesting;
+class Panel;

[jianli, 2011/08/09 21:10:23]

We already include panel.h. I think this is removed in jennb's patch.

[prasadt, 2011/08/09 23:05:04]

Done.

 
 namespace gfx {
 class Rect;
@@ -62,18 +63,19 @@ class NativePanel {
   virtual bool IsDrawingAttention() const = 0;
   virtual Browser* GetPanelBrowser() const = 0;
   virtual void DestroyPanelBrowser() = 0;
-
-  // Returns a pointer to the testing interface to the native panel.
-  virtual NativePanelTesting* GetNativePanelTesting() = 0;
 };
 
 // A NativePanel utility interface used for accessing elements of the
 // native panel used only by test automation.
 class NativePanelTesting {
  public:
+  static NativePanelTesting* Create(NativePanel* native_panel);
 
- protected:
-  virtual ~NativePanelTesting() {}
+  virtual void TitlebarLeftButtonPress(gfx::Point point) = 0;

[jianli, 2011/08/09 21:10:23]

It would be better to call it either TitlebarLeftButtonPressed or
PressTitlebarLeftButton.

Also, button seems to be vague since it might mean the mouse button or the UI
button. How about calling it TitlebarLeftMouseButtonPressed.

One other thing is about we should make bar capitalized. It seems that we have
mixed use of both Titlebar and TitleBar.

[prasadt, 2011/08/09 23:05:04]

I changed the name to PressLeftMouseButtonTitlebar.  Keeping Titlebar to the end
- I think its makes better grouping of the functions.

TitlebarLeftMouseButtonPressed sounds like an event handler than a verb. 
Titlebar vs TitleBar, you're right we're mixing them up.  I picked Titlebar, I
think of it as one word. 

+  virtual void TitlebarLeftButtonRelease() = 0;
+  virtual void TitlebarDrag(int delta_x, int delta_y) = 0;
+  virtual void TitlebarCancelDrag() = 0;
+  virtual void TitlebarFinishDrag() = 0;
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_NATIVE_PANEL_H_