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

Unified Diff: chrome/browser/ui/extensions/application_launch.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/extensions/application_launch.cc
diff --git a/chrome/browser/ui/extensions/application_launch.cc b/chrome/browser/ui/extensions/application_launch.cc
index 74ce425b27a4c7a6b53dae6f9dbbeb69348873bd..8bd010280b84b32e04761a7394eafb32a3581c3c 100644
--- a/chrome/browser/ui/extensions/application_launch.cc
+++ b/chrome/browser/ui/extensions/application_launch.cc
@@ -72,8 +72,7 @@ class EnableViaAppListFlow : public ExtensionEnableFlowDelegate {
callback_(callback) {
}
- virtual ~EnableViaAppListFlow() {
- }
+ ~EnableViaAppListFlow() override {}
void Run() {
DCHECK(!service_->IsExtensionEnabled(extension_id_));
@@ -90,7 +89,7 @@ class EnableViaAppListFlow : public ExtensionEnableFlowDelegate {
}
// ExtensionEnableFlowDelegate overrides.
- virtual void ExtensionEnableFlowFinished() override {
+ void ExtensionEnableFlowFinished() override {
const Extension* extension =
service_->GetExtensionById(extension_id_, false);
if (!extension)
@@ -99,9 +98,7 @@ class EnableViaAppListFlow : public ExtensionEnableFlowDelegate {
delete this;
}
- virtual void ExtensionEnableFlowAborted(bool user_initiated) override {
- delete this;
- }
+ void ExtensionEnableFlowAborted(bool user_initiated) override { delete this; }
ExtensionService* service_;
Profile* profile_;

Powered by Google App Engine
This is Rietveld 408576698