Action box menu

chrome/browser/ui/views/browser_actions_host_delegate.h

+// Copyright (c) 2012 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_VIEWS_BROWSER_ACTIONS_HOST_DELEGATE_H_
+#define CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_HOST_DELEGATE_H_
+#pragma once
+
+#include "ui/gfx/size.h"
+#include "ui/views/drag_controller.h"
+
+class BrowserActionButton;
+class Browser;
+
+namespace views {
+class FocusManager;
+}
+
+
+class BrowserActionsHostDelegate : public views::DragController {

[Aaron Boodman, 2012/06/12 05:53:44]

Can this be BrowserActionView::Delegate? That would be the typical convention in
Chrome.

[yefimt, 2012/06/13 01:24:21]

Done.

+ public:
+  virtual Browser* GetBrowser() const = 0;
+  virtual int GetCurrentTabId() const = 0;
+  virtual void OnBrowserActionExecuted(BrowserActionButton* button) = 0;
+  virtual void OnBrowserActionVisibilityChanged() = 0;

[Aaron Boodman, 2012/06/12 05:53:44]

This one is not used. Seems like something only the toolbar would care about,
since visibility cannot change for action box.

[yefimt, 2012/06/13 01:24:21]

this delegate is used by both menu and toolbar

On 2012/06/12 05:53:44, Aaron Boodman wrote:

+  virtual gfx::Size GetContentOffset() const = 0;

[Aaron Boodman, 2012/06/12 05:53:44]

This needs a better name.

[yefimt, 2012/06/13 01:24:21]

Renamed to GetViewContentOffset()
On 2012/06/12 05:53:44, Aaron Boodman wrote:

+
+
+ protected:
+  BrowserActionsHostDelegate() {}
+  virtual ~BrowserActionsHostDelegate() {}
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_BROWSER_ACTIONS_HOST_DELEGATE_H_