Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7343)

Unified Diff: chrome/browser/ui/views/extensions/extension_action_view_delegate.h

Issue 661493004: Add infrastructure for Chrome Actions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/extensions/extension_action_view_delegate.h
diff --git a/chrome/browser/ui/views/extensions/extension_action_view_delegate.h b/chrome/browser/ui/views/extensions/extension_action_view_delegate.h
deleted file mode 100644
index 0b445c0fd6be7ecdee90d80f4c70fd60131147de..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/extensions/extension_action_view_delegate.h
+++ /dev/null
@@ -1,71 +0,0 @@
-// Copyright 2014 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_EXTENSIONS_EXTENSION_ACTION_VIEW_DELEGATE_H_
-#define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_ACTION_VIEW_DELEGATE_H_
-
-namespace content {
-class WebContents;
-}
-
-namespace views {
-class FocusManager;
-class MenuButton;
-class View;
-class Widget;
-}
-
-class ExtensionActionViewController;
-
-// The view that surrounds an ExtensionAction and owns the
-// ExtensionActionViewController. Since different actions can subclass
-// different views, we don't derive views::View directly here.
-class ExtensionActionViewDelegate {
- public:
- // Returns |this| as a view. We need this because our subclasses implement
- // different kinds of views, and inheriting View here is a really bad idea.
- virtual views::View* GetAsView() = 0;
-
- // Returns true if this view is being shown inside a menu.
- virtual bool IsShownInMenu() = 0;
-
- // Returns the FocusManager to use when registering accelerators.
- virtual views::FocusManager* GetFocusManagerForAccelerator() = 0;
-
- // Returns the parent for the associated context menu.
- virtual views::Widget* GetParentForContextMenu() = 0;
-
- // In some cases (such as when an action is shown in a menu), a substitute
- // ExtensionActionViewController should be used for showing popups. This
- // returns the preferred control.
- virtual ExtensionActionViewController* GetPreferredPopupViewController() = 0;
-
- // Returns the reference view for the extension action's popup.
- virtual views::View* GetReferenceViewForPopup() = 0;
-
- // Returns the MenuButton (if any) to use in showing the context menu (this
- // allows views code to update the pressed state of the button).
- virtual views::MenuButton* GetContextMenuButton() = 0;
-
- // Returns the current web contents.
- virtual content::WebContents* GetCurrentWebContents() = 0;
-
- // Hides whatever popup is active (even if it's not this one).
- virtual void HideActivePopup() = 0;
-
- // Called when the icon is updated; this is forwarded from the icon factory.
- virtual void OnIconUpdated() = 0;
-
- // Called when a popup is shown. See ExecuteAction() for the definition of
- // |grant_tab_permissions|.
- virtual void OnPopupShown(bool grant_tab_permissions) {}
-
- // Does any additional cleanup after the popup is closed.
- virtual void CleanupPopup() {}
-
- protected:
- virtual ~ExtensionActionViewDelegate() {}
-};
-
-#endif // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_ACTION_VIEW_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698