Standardize usage of virtual/override/final specifiers.

The Google C++ style guide states:

  Explicitly annotate overrides of virtual functions or virtual
  destructors with an override or (less frequently) final specifier.
  Older (pre-C++11) code will use the virtual keyword as an inferior
  alternative annotation. For clarity, use exactly one of override,
  final, or virtual when declaring an override.

To better conform to these guidelines, the following constructs have
been rewritten:

- if a base class has a virtual destructor, then:
    virtual ~Foo();                   ->  ~Foo() override;
- virtual void Foo() override;        ->  void Foo() override;
- virtual void Foo() override final;  ->  void Foo() final;

This patch was automatically generated. The clang plugin can generate
fixit hints, which are suggested edits when it is 100% sure it knows how
to fix a problem. The hints from the clang plugin were applied to the
source tree using the tool in https://codereview.chromium.org/598073004.

BUG=417463
R=rch@chromium.org

Review URL: https://codereview.chromium.org/678073004

Cr-Commit-Position: refs/heads/master@{#301457}

3 files changed, 35 insertions, 38 deletions

diff --git a/net/tools/balsa/balsa_frame_test.cc b/net/tools/balsa/balsa_frame_test.cc
index 334f5ef..4a00a66 100644
--- a/net/tools/balsa/balsa_frame_test.cc
+++ b/net/tools/balsa/balsa_frame_test.cc
@@ -58,7 +58,7 @@ class Visitor : public BalsaVisitorInterface {
 
 class BalsaFrameTest : public ::testing::Test {
  public:
-  virtual void SetUp() override {
+  void SetUp() override {
     frame_.reset(new BalsaFrame);
     frame_headers_.reset(new BalsaHeaders);
     visitor_.reset(new Visitor);
diff --git a/net/tools/balsa/balsa_headers_test.cc b/net/tools/balsa/balsa_headers_test.cc
index b1479c6..3dc8eb3 100644
--- a/net/tools/balsa/balsa_headers_test.cc
+++ b/net/tools/balsa/balsa_headers_test.cc
@@ -19,7 +19,7 @@ using ::base::StringPiece;
 
 class BalsaBufferTest : public ::testing::Test {
  public:
-  virtual void SetUp() override {
+  void SetUp() override {
     buffer_.reset(new BalsaBuffer);
     anotherBuffer_.reset(new BalsaBuffer);
   }
@@ -33,9 +33,7 @@ namespace {
 
 class BalsaHeadersTest: public ::testing::Test {
  public:
-  virtual void SetUp() override {
-    headers_.reset(new BalsaHeaders);
-  }
+  void SetUp() override { headers_.reset(new BalsaHeaders); }
 
  protected:
   scoped_ptr<BalsaHeaders> headers_;
diff --git a/net/tools/balsa/noop_balsa_visitor.h b/net/tools/balsa/noop_balsa_visitor.h
index c47b0b3..89e9719 100644
--- a/net/tools/balsa/noop_balsa_visitor.h
+++ b/net/tools/balsa/noop_balsa_visitor.h
@@ -17,39 +17,38 @@ namespace net {
 class NoOpBalsaVisitor : public BalsaVisitorInterface {
  public:
   NoOpBalsaVisitor() { }
-  virtual ~NoOpBalsaVisitor() { }
-
-  virtual void ProcessBodyInput(const char* input, size_t size) override { }
-  virtual void ProcessBodyData(const char* input, size_t size) override { }
-  virtual void ProcessHeaderInput(const char* input, size_t size) override { }
-  virtual void ProcessTrailerInput(const char* input, size_t size) override { }
-  virtual void ProcessHeaders(const BalsaHeaders& headers) override { }
-
-  virtual void ProcessRequestFirstLine(const char* line_input,
-                                       size_t line_length,
-                                       const char* method_input,
-                                       size_t method_length,
-                                       const char* request_uri_input,
-                                       size_t request_uri_length,
-                                       const char* version_input,
-                                       size_t version_length) override { }
-  virtual void ProcessResponseFirstLine(const char* line_input,
-                                        size_t line_length,
-                                        const char* version_input,
-                                        size_t version_length,
-                                        const char* status_input,
-                                        size_t status_length,
-                                        const char* reason_input,
-                                        size_t reason_length) override { }
-  virtual void ProcessChunkLength(size_t chunk_length) override { }
-  virtual void ProcessChunkExtensions(const char* input, size_t size) override {
-  }
-  virtual void HeaderDone() override { }
-  virtual void MessageDone() override { }
-  virtual void HandleHeaderError(BalsaFrame* framer) override { }
-  virtual void HandleHeaderWarning(BalsaFrame* framer) override { }
-  virtual void HandleChunkingError(BalsaFrame* framer) override { }
-  virtual void HandleBodyError(BalsaFrame* framer) override { }
+  ~NoOpBalsaVisitor() override {}
+
+  void ProcessBodyInput(const char* input, size_t size) override {}
+  void ProcessBodyData(const char* input, size_t size) override {}
+  void ProcessHeaderInput(const char* input, size_t size) override {}
+  void ProcessTrailerInput(const char* input, size_t size) override {}
+  void ProcessHeaders(const BalsaHeaders& headers) override {}
+
+  void ProcessRequestFirstLine(const char* line_input,
+                               size_t line_length,
+                               const char* method_input,
+                               size_t method_length,
+                               const char* request_uri_input,
+                               size_t request_uri_length,
+                               const char* version_input,
+                               size_t version_length) override {}
+  void ProcessResponseFirstLine(const char* line_input,
+                                size_t line_length,
+                                const char* version_input,
+                                size_t version_length,
+                                const char* status_input,
+                                size_t status_length,
+                                const char* reason_input,
+                                size_t reason_length) override {}
+  void ProcessChunkLength(size_t chunk_length) override {}
+  void ProcessChunkExtensions(const char* input, size_t size) override {}
+  void HeaderDone() override {}
+  void MessageDone() override {}
+  void HandleHeaderError(BalsaFrame* framer) override {}
+  void HandleHeaderWarning(BalsaFrame* framer) override {}
+  void HandleChunkingError(BalsaFrame* framer) override {}
+  void HandleBodyError(BalsaFrame* framer) override {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NoOpBalsaVisitor);