platform2: Replace 'OVERRIDE' with 'override' and fix linter
Replace OVERRIDE macro with native C++ 'override' keyword.
Also fixed the style issue of using both 'override' and 'virtual'
on member function overrides. Only one is required and cpplint
complains if both are specified now.
BUG=None
TEST=Build all affected target and ran unit tests which passed.
Change-Id: I04474aacad2ca8d6c6042c84a5378961d5bd8bf2
Reviewed-on: https://chromium-review.googlesource.com/212180
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
diff --git a/crash_reporter/chrome_collector_test.cc b/crash_reporter/chrome_collector_test.cc
index 843ca8e..05e8f2d 100644
--- a/crash_reporter/chrome_collector_test.cc
+++ b/crash_reporter/chrome_collector_test.cc
@@ -51,7 +51,7 @@
ChromeCollector collector_;
private:
- void SetUp() OVERRIDE {
+ void SetUp() override {
collector_.Initialize(CountCrash, IsMetrics);
chromeos::ClearLog();
}
diff --git a/crash_reporter/kernel_collector_test.cc b/crash_reporter/kernel_collector_test.cc
index ec7ad8e..d6414a6 100644
--- a/crash_reporter/kernel_collector_test.cc
+++ b/crash_reporter/kernel_collector_test.cc
@@ -46,7 +46,7 @@
KernelCollector collector_;
private:
- void SetUp() OVERRIDE {
+ void SetUp() override {
s_crashes = 0;
s_metrics = true;
collector_.Initialize(CountCrash, IsMetrics);
diff --git a/crash_reporter/udev_collector_test.cc b/crash_reporter/udev_collector_test.cc
index b45fdd8..5797854 100644
--- a/crash_reporter/udev_collector_test.cc
+++ b/crash_reporter/udev_collector_test.cc
@@ -55,7 +55,7 @@
}
private:
- void SetUp() OVERRIDE {
+ void SetUp() override {
s_consent_given = true;
collector_.Initialize(CountCrash, IsMetrics);