Standardize usage of virtual/override/final specifiers.

content/browser/power_monitor_message_broadcaster_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/test/power_monitor_test_base.h"
 #include "content/browser/power_monitor_message_broadcaster.h"
 #include "content/common/power_monitor_messages.h"
 #include "ipc/ipc_sender.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 35 matching lines @@
   int resumes_;  // Count of OnResume notifications.
 };
 
 class PowerMonitorMessageBroadcasterTest : public testing::Test {
  protected:
   PowerMonitorMessageBroadcasterTest() {
     power_monitor_source_ = new base::PowerMonitorTestSource();
     power_monitor_.reset(new base::PowerMonitor(
         scoped_ptr<base::PowerMonitorSource>(power_monitor_source_)));
   }
-  virtual ~PowerMonitorMessageBroadcasterTest() {};
+  ~PowerMonitorMessageBroadcasterTest() override{};

[nasko, 2014/10/27 21:06:39]

nit: shouldn't there be space between override and {}?

[dcheng, 2014/10/27 22:00:12]

Oh. It's because of the superfluous semicolon. I'll file a clang-format bug for
this, though I'd argue this is pretty poor style to begin with.

 
   base::PowerMonitorTestSource* source() { return power_monitor_source_; }
   base::PowerMonitor* monitor() { return power_monitor_.get(); }
 
  private:
   base::PowerMonitorTestSource* power_monitor_source_;
   scoped_ptr<base::PowerMonitor> power_monitor_;
 
   DISALLOW_COPY_AND_ASSIGN(PowerMonitorMessageBroadcasterTest);
 };
@@ skipping 37 matching lines @@
   // Pretend the device has gone off battery power
   source()->GeneratePowerStateEvent(false);
   EXPECT_EQ(sender.power_state_changes(), 3);
 
   // Repeated indications the device is off battery power should be suppressed.
   source()->GeneratePowerStateEvent(false);
   EXPECT_EQ(sender.power_state_changes(), 3);
 }
 
 }  // namespace base