Standardize usage of virtual/override/final in ipc/

ipc/ipc_perftest_support.cc

 // Copyright (c) 2012 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 "ipc/ipc_perftest_support.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 63 matching lines @@
 // message. It assumes each message has one string parameter. When the string
 // "quit" is sent, it will exit.
 class ChannelReflectorListener : public Listener {
  public:
   ChannelReflectorListener()
       : channel_(NULL),
         latency_tracker_("Client messages") {
     VLOG(1) << "Client listener up";
   }
 
-  virtual ~ChannelReflectorListener() {
+  ~ChannelReflectorListener() override {
     VLOG(1) << "Client listener down";
     latency_tracker_.ShowResults();
   }
 
   void Init(Channel* channel) {
     DCHECK(!channel_);
     channel_ = channel;
   }
 
-  virtual bool OnMessageReceived(const Message& message) override {
+  bool OnMessageReceived(const Message& message) override {
     CHECK(channel_);
 
     PickleIterator iter(message);
     int64 time_internal;
     EXPECT_TRUE(iter.ReadInt64(&time_internal));
     int msgid;
     EXPECT_TRUE(iter.ReadInt(&msgid));
     std::string payload;
     EXPECT_TRUE(iter.ReadString(&payload));
 
@@ skipping 30 matching lines @@
   explicit PerformanceChannelListener(const std::string& label)
       : label_(label),
         sender_(NULL),
         msg_count_(0),
         msg_size_(0),
         count_down_(0),
         latency_tracker_("Server messages") {
     VLOG(1) << "Server listener up";
   }
 
-  virtual ~PerformanceChannelListener() {
-    VLOG(1) << "Server listener down";
-  }
+  ~PerformanceChannelListener() override { VLOG(1) << "Server listener down"; }

[agl, 2014/10/21 17:49:49]

Please don't make functions single line just because they can be.

[dcheng, 2014/10/21 17:57:44]

This is a clang-format change. I'm willing to make manual formatting changes for
this patch, but in general, this is not possible with large automated rewrites.

 
   void Init(Sender* sender) {
     DCHECK(!sender_);
     sender_ = sender;
   }
 
   // Call this before running the message loop.
   void SetTestParams(int msg_count, size_t msg_size) {
     DCHECK_EQ(0, count_down_);
     msg_count_ = msg_count;
     msg_size_ = msg_size;
     count_down_ = msg_count_;
     payload_ = std::string(msg_size_, 'a');
   }
 
-  virtual bool OnMessageReceived(const Message& message) override {
+  bool OnMessageReceived(const Message& message) override {
     CHECK(sender_);
 
     PickleIterator iter(message);
     int64 time_internal;
     EXPECT_TRUE(iter.ReadInt64(&time_internal));
     int msgid;
     EXPECT_TRUE(iter.ReadInt(&msgid));
     std::string reflected_payload;
     EXPECT_TRUE(iter.ReadString(&reflected_payload));
 
@@ skipping 159 matching lines @@
   base::MessageLoop::current()->Run();
   return 0;
 }
 
 scoped_refptr<base::TaskRunner> PingPongTestClient::task_runner() {
   return main_message_loop_.message_loop_proxy();
 }
 
 }  // namespace test
 }  // namespace IPC