Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0dfc9ceb authored by Alex Vakulenko's avatar Alex Vakulenko Committed by chrome-internal-fetch
Browse files

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: default avatarAlex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: default avatarAlex Vakulenko <avakulenko@chromium.org>
parent 334b1ae2

crash_reporter/chrome_collector_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -51,7 +51,7 @@ class ChromeCollectorTest : public ::testing::Test { 
  ChromeCollector collector_;



 private:

  void SetUp() OVERRIDE {

  void SetUp() override {

    collector_.Initialize(CountCrash, IsMetrics);

    chromeos::ClearLog();

  }

crash_reporter/kernel_collector_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -46,7 +46,7 @@ class KernelCollectorTest : public ::testing::Test { 
  KernelCollector collector_;



 private:

  void SetUp() OVERRIDE {

  void SetUp() override {

    s_crashes = 0;

    s_metrics = true;

    collector_.Initialize(CountCrash, IsMetrics);

crash_reporter/udev_collector_test.cc

+1 −1
Original line number Diff line number Diff line
@@ -55,7 +55,7 @@ class UdevCollectorTest : public ::testing::Test { 
  }



 private:

  void SetUp() OVERRIDE {

  void SetUp() override {

    s_consent_given = true;



    collector_.Initialize(CountCrash, IsMetrics);