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

Unified Diff: components/policy/core/common/cloud/cloud_policy_manager_unittest.cc

Issue 666133002: Standardize usage of virtual/override/final in components/ (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: components/policy/core/common/cloud/cloud_policy_manager_unittest.cc
diff --git a/components/policy/core/common/cloud/cloud_policy_manager_unittest.cc b/components/policy/core/common/cloud/cloud_policy_manager_unittest.cc
index 41d7a907f734f5ac5381c564ba3fbcf0788634bb..1d6f0531df9ff961599487d91ea4c5c32356ee56 100644
--- a/components/policy/core/common/cloud/cloud_policy_manager_unittest.cc
+++ b/components/policy/core/common/cloud/cloud_policy_manager_unittest.cc
@@ -31,25 +31,24 @@ namespace {
class TestHarness : public PolicyProviderTestHarness {
public:
explicit TestHarness(PolicyLevel level);
- virtual ~TestHarness();
+ ~TestHarness() override;
- virtual void SetUp() override;
+ void SetUp() override;
- virtual ConfigurationPolicyProvider* CreateProvider(
+ ConfigurationPolicyProvider* CreateProvider(
SchemaRegistry* registry,
scoped_refptr<base::SequencedTaskRunner> task_runner) override;
- virtual void InstallEmptyPolicy() override;
- virtual void InstallStringPolicy(const std::string& policy_name,
- const std::string& policy_value) override;
- virtual void InstallIntegerPolicy(const std::string& policy_name,
- int policy_value) override;
- virtual void InstallBooleanPolicy(const std::string& policy_name,
- bool policy_value) override;
- virtual void InstallStringListPolicy(
- const std::string& policy_name,
- const base::ListValue* policy_value) override;
- virtual void InstallDictionaryPolicy(
+ void InstallEmptyPolicy() override;
+ void InstallStringPolicy(const std::string& policy_name,
+ const std::string& policy_value) override;
+ void InstallIntegerPolicy(const std::string& policy_name,
+ int policy_value) override;
+ void InstallBooleanPolicy(const std::string& policy_name,
+ bool policy_value) override;
+ void InstallStringListPolicy(const std::string& policy_name,
+ const base::ListValue* policy_value) override;
+ void InstallDictionaryPolicy(
const std::string& policy_name,
const base::DictionaryValue* policy_value) override;
@@ -156,7 +155,7 @@ class TestCloudPolicyManager : public CloudPolicyManager {
task_runner,
task_runner,
task_runner) {}
- virtual ~TestCloudPolicyManager() {}
+ ~TestCloudPolicyManager() override {}
// Publish the protected members for testing.
using CloudPolicyManager::client;

Powered by Google App Engine
This is Rietveld 408576698