Fix Clang-plugin style error in ChildBrokerHost

mojo/edk/system/child_broker_host.h

 // Copyright 2015 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.
 
 #ifndef MOJO_EDK_SYSTEM_CHILD_BROKER_HOST_H_
 #define MOJO_EDK_SYSTEM_CHILD_BROKER_HOST_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 16 matching lines @@
 #endif
  public:
   // |child_process| is a handle to the child process. It's not owned by this
   // class but is guaranteed to be alive as long as the child process is
   // running. |pipe| is a handle to the communication pipe to the child process,
   // which is generated inside mojo::edk::ChildProcessLaunched. It is owned by
   // this class.
   ChildBrokerHost(base::ProcessHandle child_process, ScopedPlatformHandle pipe);
 
  private:
+#if defined(OS_WIN)
+  ~ChildBrokerHost() override;
+#else
   virtual ~ChildBrokerHost();

[jam, 2015/11/25 22:03:47]

no need for virtual actually

+#endif
 
   void RegisterIOHandler();
   void BeginRead();
 
 #if defined(OS_WIN)
   void OnIOCompleted(base::MessageLoopForIO::IOContext* context,
                      DWORD bytes_transferred,
                      DWORD error) override;
 
   // Helper wrappers around DuplicateHandle.
@@ skipping 12 matching lines @@
   std::vector<char> read_data_;
   // How many bytes in read_data_ we already read.
   uint32_t num_bytes_read_;
   std::vector<char> write_data_;
 };
 
 }  // namespace edk
 }  // namespace mojo
 
 #endif  // MOJO_EDK_SYSTEM_CHILD_BROKER_HOST_H_