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

Unified Diff: chrome/browser/ui/window_sizer/window_sizer.cc

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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/window_sizer/window_sizer.cc
diff --git a/chrome/browser/ui/window_sizer/window_sizer.cc b/chrome/browser/ui/window_sizer/window_sizer.cc
index c1ce2739072aa537f1fb0aa05e8eab42a21100a6..77a9eab1bdbf5e5d23a834460ee92b45ea5f9d12 100644
--- a/chrome/browser/ui/window_sizer/window_sizer.cc
+++ b/chrome/browser/ui/window_sizer/window_sizer.cc
@@ -42,10 +42,9 @@ class DefaultStateProvider : public WindowSizer::StateProvider {
}
// Overridden from WindowSizer::StateProvider:
- virtual bool GetPersistentState(
- gfx::Rect* bounds,
- gfx::Rect* work_area,
- ui::WindowShowState* show_state) const override {
+ bool GetPersistentState(gfx::Rect* bounds,
+ gfx::Rect* work_area,
+ ui::WindowShowState* show_state) const override {
DCHECK(bounds);
DCHECK(show_state);
@@ -85,7 +84,7 @@ class DefaultStateProvider : public WindowSizer::StateProvider {
return has_prefs;
}
- virtual bool GetLastActiveWindowState(
+ bool GetLastActiveWindowState(
gfx::Rect* bounds,
ui::WindowShowState* show_state) const override {
DCHECK(show_state);
@@ -141,11 +140,10 @@ class DefaultStateProvider : public WindowSizer::StateProvider {
class DefaultTargetDisplayProvider : public WindowSizer::TargetDisplayProvider {
public:
DefaultTargetDisplayProvider() {}
- virtual ~DefaultTargetDisplayProvider() {}
+ ~DefaultTargetDisplayProvider() override {}
- virtual gfx::Display GetTargetDisplay(
- const gfx::Screen* screen,
- const gfx::Rect& bounds) const override {
+ gfx::Display GetTargetDisplay(const gfx::Screen* screen,
+ const gfx::Rect& bounds) const override {
#if defined(USE_ASH)
bool force_ash = false;
// On Windows check if the browser is launched to serve ASH. If yes then

Powered by Google App Engine
This is Rietveld 408576698