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

Unified Diff: chrome/browser/ui/views/frame/immersive_mode_controller_ash.h

Issue 686463004: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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/frame/immersive_mode_controller_ash.h
diff --git a/chrome/browser/ui/views/frame/immersive_mode_controller_ash.h b/chrome/browser/ui/views/frame/immersive_mode_controller_ash.h
index 005bb715cbdc83f1ba87bd84ccdff7d54838a455..16fb17437dfe60908999a4d4b97bea290e5b1858 100644
--- a/chrome/browser/ui/views/frame/immersive_mode_controller_ash.h
+++ b/chrome/browser/ui/views/frame/immersive_mode_controller_ash.h
@@ -25,22 +25,22 @@ class ImmersiveModeControllerAsh
public content::NotificationObserver {
public:
ImmersiveModeControllerAsh();
- virtual ~ImmersiveModeControllerAsh();
+ ~ImmersiveModeControllerAsh() override;
// ImmersiveModeController overrides:
- virtual void Init(BrowserView* browser_view) override;
- virtual void SetEnabled(bool enabled) override;
- virtual bool IsEnabled() const override;
- virtual bool ShouldHideTabIndicators() const override;
- virtual bool ShouldHideTopViews() const override;
- virtual bool IsRevealed() const override;
- virtual int GetTopContainerVerticalOffset(
+ void Init(BrowserView* browser_view) override;
+ void SetEnabled(bool enabled) override;
+ bool IsEnabled() const override;
+ bool ShouldHideTabIndicators() const override;
+ bool ShouldHideTopViews() const override;
+ bool IsRevealed() const override;
+ int GetTopContainerVerticalOffset(
const gfx::Size& top_container_size) const override;
- virtual ImmersiveRevealedLock* GetRevealedLock(
- AnimateReveal animate_reveal) override WARN_UNUSED_RESULT;
- virtual void OnFindBarVisibleBoundsChanged(
+ ImmersiveRevealedLock* GetRevealedLock(AnimateReveal animate_reveal) override
+ WARN_UNUSED_RESULT;
+ void OnFindBarVisibleBoundsChanged(
const gfx::Rect& new_visible_bounds_in_screen) override;
- virtual void SetupForTest() override;
+ void SetupForTest() override;
private:
// Enables or disables observers for window restore and entering / exiting
@@ -55,21 +55,20 @@ class ImmersiveModeControllerAsh
bool UpdateTabIndicators();
// ImmersiveFullscreenController::Delegate overrides:
- virtual void OnImmersiveRevealStarted() override;
- virtual void OnImmersiveRevealEnded() override;
- virtual void OnImmersiveFullscreenExited() override;
- virtual void SetVisibleFraction(double visible_fraction) override;
- virtual std::vector<gfx::Rect> GetVisibleBoundsInScreen() const override;
+ void OnImmersiveRevealStarted() override;
+ void OnImmersiveRevealEnded() override;
+ void OnImmersiveFullscreenExited() override;
+ void SetVisibleFraction(double visible_fraction) override;
+ std::vector<gfx::Rect> GetVisibleBoundsInScreen() const override;
// ash::wm::WindowStateObserver override:
- virtual void OnPostWindowStateTypeChange(
- ash::wm::WindowState* window_state,
- ash::wm::WindowStateType old_type) override;
+ void OnPostWindowStateTypeChange(ash::wm::WindowState* window_state,
+ ash::wm::WindowStateType old_type) override;
// content::NotificationObserver override:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) override;
+ void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) override;
scoped_ptr<ash::ImmersiveFullscreenController> controller_;
« no previous file with comments | « chrome/browser/ui/views/frame/global_menu_bar_x11.h ('k') | chrome/browser/ui/views/frame/immersive_mode_controller_stub.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698