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

Unified Diff: chrome/installer/util/firewall_manager_win.cc

Issue 627423002: Replacing the OVERRIDE with override in chrome/installer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Patch is rebased 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/installer/util/firewall_manager_win.cc
diff --git a/chrome/installer/util/firewall_manager_win.cc b/chrome/installer/util/firewall_manager_win.cc
index 54142ffa1487ae644dc2e4a3b74979b0c72a52f4..3ec15b57f18c5412a79560a76223d3a4f36612a0 100644
--- a/chrome/installer/util/firewall_manager_win.cc
+++ b/chrome/installer/util/firewall_manager_win.cc
@@ -30,16 +30,16 @@ class FirewallManagerAdvancedImpl : public FirewallManager {
}
// FirewallManager methods.
- virtual bool CanUseLocalPorts() OVERRIDE {
+ virtual bool CanUseLocalPorts() override {
return !manager_.IsFirewallEnabled() || manager_.HasAnyRule();
};
- virtual bool AddFirewallRules() OVERRIDE {
+ virtual bool AddFirewallRules() override {
return manager_.AddUDPRule(GetMdnsRuleName(), GetMdnsRuleDescription(),
kDefaultMdnsPort);
}
- virtual void RemoveFirewallRules() OVERRIDE {
+ virtual void RemoveFirewallRules() override {
manager_.DeleteAllRules();
}
@@ -74,18 +74,18 @@ class FirewallManagerLegacyImpl : public FirewallManager {
}
// FirewallManager methods.
- virtual bool CanUseLocalPorts() OVERRIDE {
+ virtual bool CanUseLocalPorts() override {
return !manager_.IsFirewallEnabled() ||
manager_.GetAllowIncomingConnection(NULL);
};
- virtual bool AddFirewallRules() OVERRIDE {
+ virtual bool AddFirewallRules() override {
// Change nothing if rule is set.
return manager_.GetAllowIncomingConnection(NULL) ||
manager_.SetAllowIncomingConnection(true);
}
- virtual void RemoveFirewallRules() OVERRIDE {
+ virtual void RemoveFirewallRules() override {
manager_.DeleteRule();
}
« no previous file with comments | « chrome/installer/util/delete_reg_key_work_item.h ('k') | chrome/installer/util/google_chrome_distribution.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698