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

Side by Side Diff: chrome/browser/ui/views/browser_action_view.h

Issue 10700007: Page actions behave as disableable browser actions when script badges are enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "chrome/browser/extensions/image_loading_tracker.h" 10 #include "chrome/browser/extensions/image_loading_tracker.h"
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 bool is_mouse_gesture) OVERRIDE; 81 bool is_mouse_gesture) OVERRIDE;
82 82
83 // Overridden from ui::AcceleratorTarget. 83 // Overridden from ui::AcceleratorTarget.
84 virtual bool AcceleratorPressed(const ui::Accelerator& accelerator) OVERRIDE; 84 virtual bool AcceleratorPressed(const ui::Accelerator& accelerator) OVERRIDE;
85 85
86 // Notifications when to set button state to pushed/not pushed (for when the 86 // Notifications when to set button state to pushed/not pushed (for when the
87 // popup/context menu is hidden or shown by the container). 87 // popup/context menu is hidden or shown by the container).
88 void SetButtonPushed(); 88 void SetButtonPushed();
89 void SetButtonNotPushed(); 89 void SetButtonNotPushed();
90 90
91 // Whether the browser action is enabled on this tab. Note that we cannot use
92 // the built-in views enabled/SetEnabled because disabled views cannot
sky 2012/07/16 15:43:46 cannot -> do not
Yoyo Zhou 2012/07/16 17:45:45 Done.
93 // receive drag events.
94 bool IsEnabled(int tab_id) const;
95
91 protected: 96 protected:
92 // Overridden from views::View: 97 // Overridden from views::View:
93 virtual void ViewHierarchyChanged(bool is_add, 98 virtual void ViewHierarchyChanged(bool is_add,
94 View* parent, 99 View* parent,
95 View* child) OVERRIDE; 100 View* child) OVERRIDE;
96 101
97 private: 102 private:
98 virtual ~BrowserActionButton(); 103 virtual ~BrowserActionButton();
99 104
100 // Register an extension command if the extension has an active one. 105 // Register an extension command if the extension has an active one.
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
167 // The container for this view. 172 // The container for this view.
168 BrowserActionsContainer* panel_; 173 BrowserActionsContainer* panel_;
169 174
170 // The button this view contains. 175 // The button this view contains.
171 BrowserActionButton* button_; 176 BrowserActionButton* button_;
172 177
173 DISALLOW_COPY_AND_ASSIGN(BrowserActionView); 178 DISALLOW_COPY_AND_ASSIGN(BrowserActionView);
174 }; 179 };
175 180
176 #endif // CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_ 181 #endif // CHROME_BROWSER_UI_VIEWS_BROWSER_ACTION_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698