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

Unified Diff: chrome/browser/chromeos/frame/browser_view.h

Issue 8438064: Separate StatusAreaView from StatusAreaViewChromeos (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 9 years, 1 month 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/frame/browser_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/frame/browser_view.h
diff --git a/chrome/browser/chromeos/frame/browser_view.h b/chrome/browser/chromeos/frame/browser_view.h
index 14bc60945b8d8fb7f2058386eca909ba95d0a303..d09e411f5ae8e8a0810ed9f678b0d8da30e37735 100644
--- a/chrome/browser/chromeos/frame/browser_view.h
+++ b/chrome/browser/chromeos/frame/browser_view.h
@@ -10,7 +10,7 @@
#include "base/compiler_specific.h"
#include "base/message_loop.h"
-#include "chrome/browser/chromeos/status/status_area_host.h"
+#include "chrome/browser/chromeos/status/status_area_button.h"
#include "chrome/browser/ui/browser_list.h"
#include "chrome/browser/ui/views/frame/browser_view.h"
#include "views/context_menu_controller.h"
@@ -18,6 +18,7 @@
class AccessibleToolbarView;
class Profile;
+class StatusAreaButton;
class TabStripModel;
namespace ui {
@@ -35,8 +36,7 @@ class MenuRunner;
namespace chromeos {
class LayoutModeButton;
-class StatusAreaView;
-class StatusAreaButton;
+class StatusAreaViewChromeos;
// chromeos::BrowserView adds ChromeOS specific controls and menus to a
// BrowserView created with Browser::TYPE_TABBED. This extender adds controls
@@ -49,7 +49,7 @@ class BrowserView : public ::BrowserView,
public views::ContextMenuController,
public views::MenuListener,
public BrowserList::Observer,
- public StatusAreaHost,
+ public StatusAreaButton::Delegate,
public MessageLoopForUI::Observer {
public:
explicit BrowserView(Browser* browser);
@@ -83,15 +83,13 @@ class BrowserView : public ::BrowserView,
virtual void OnBrowserAdded(const Browser* browser) OVERRIDE;
virtual void OnBrowserRemoved(const Browser* browser) OVERRIDE;
- // StatusAreaHost overrides.
- virtual Profile* GetProfile() const OVERRIDE;
- virtual gfx::NativeWindow GetNativeWindow() const OVERRIDE;
- virtual bool ShouldOpenButtonOptions(
- const views::View* button_view) const OVERRIDE;
- virtual void ExecuteBrowserCommand(int id) const OVERRIDE;
- virtual void OpenButtonOptions(const views::View* button_view) OVERRIDE;
- virtual ScreenMode GetScreenMode() const OVERRIDE;
- virtual TextStyle GetTextStyle() const OVERRIDE;
+ // StatusAreaButton::Delegate overrides.
+ virtual bool ShouldExecuteStatusAreaCommand(
+ const views::View* button_view, int command_id) const OVERRIDE;
+ virtual void ExecuteStatusAreaCommand(
+ const views::View* button_view, int command_id) OVERRIDE;
+ virtual gfx::Font GetStatusAreaFont(const gfx::Font& font) const OVERRIDE;
+ virtual StatusAreaButton::TextStyle GetStatusAreaTextStyle() const OVERRIDE;
virtual void ButtonVisibilityChanged(views::View* button_view) OVERRIDE;
// MessageLoopForUI::Observer overrides.
@@ -136,7 +134,7 @@ class BrowserView : public ::BrowserView,
// onscreen until Layout() is called.
void UpdateLayoutModeButtonVisibility();
- StatusAreaView* status_area_;
+ StatusAreaViewChromeos* status_area_;
LayoutModeButton* layout_mode_button_;
// System menu.
« no previous file with comments | « no previous file | chrome/browser/chromeos/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698