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

Unified Diff: components/constrained_window/constrained_window_views.cc

Issue 684513002: 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: components/constrained_window/constrained_window_views.cc
diff --git a/components/constrained_window/constrained_window_views.cc b/components/constrained_window/constrained_window_views.cc
index 43d48307336927b8f9a16aebd5006856c265a4e6..68664fbe93579fbd3700a0f7270996df382602ef 100644
--- a/components/constrained_window/constrained_window_views.cc
+++ b/components/constrained_window/constrained_window_views.cc
@@ -43,7 +43,7 @@ class WidgetModalDialogHostObserverViews
target_widget_->AddObserver(this);
}
- virtual ~WidgetModalDialogHostObserverViews() {
+ ~WidgetModalDialogHostObserverViews() override {
if (host_)
host_->RemoveObserver(this);
target_widget_->RemoveObserver(this);
@@ -51,16 +51,14 @@ class WidgetModalDialogHostObserverViews
}
// WidgetObserver overrides
- virtual void OnWidgetClosing(views::Widget* widget) override {
- delete this;
- }
+ void OnWidgetClosing(views::Widget* widget) override { delete this; }
// WebContentsModalDialogHostObserver overrides
- virtual void OnPositionRequiresUpdate() override {
+ void OnPositionRequiresUpdate() override {
UpdateWidgetModalDialogPosition(target_widget_, host_);
}
- virtual void OnHostDestroying() override {
+ void OnHostDestroying() override {
host_->RemoveObserver(this);
host_ = NULL;
}

Powered by Google App Engine
This is Rietveld 408576698