Revert of Standardize usage of virtual/override/final in ipc/ (patchset #1 id:1 of https://codereview.chromium.org/645623006/)
Reason for revert:
Addressing reviewer comments in a followup patch.
Original issue's description:
> Standardize usage of virtual/override/final in ipc/
>
> This patch was automatically generated by applying clang fixit hints
> generated by the plugin to the source tree.
>
> BUG=417463
> TBR=agl@chromium.org
>
> Committed: https://crrev.com/39be52ebbd6681c7d79ad63e2a0ab028075af0c2
> Cr-Commit-Position: refs/heads/master@{#300481}
TBR=agl@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=417463
Review URL: https://codereview.chromium.org/669953003
Cr-Commit-Position: refs/heads/master@{#300521}
CrOS-Libchrome-Original-Commit: f3076af7e0dfc078e65039a0e6faebf9a7628a63
diff --git a/ipc/mojo/ipc_channel_mojo.cc b/ipc/mojo/ipc_channel_mojo.cc
index a93adf1..a566341 100644
--- a/ipc/mojo/ipc_channel_mojo.cc
+++ b/ipc/mojo/ipc_channel_mojo.cc
@@ -27,9 +27,11 @@
Channel::Mode mode)
: delegate_(delegate), channel_handle_(channel_handle), mode_(mode) {}
- std::string GetName() const override { return channel_handle_.name; }
+ virtual std::string GetName() const override {
+ return channel_handle_.name;
+ }
- scoped_ptr<Channel> BuildChannel(Listener* listener) override {
+ virtual scoped_ptr<Channel> BuildChannel(Listener* listener) override {
return ChannelMojo::Create(delegate_, channel_handle_, mode_, listener);
}
diff --git a/ipc/mojo/ipc_channel_mojo.h b/ipc/mojo/ipc_channel_mojo.h
index e5a4e33..b307fd4 100644
--- a/ipc/mojo/ipc_channel_mojo.h
+++ b/ipc/mojo/ipc_channel_mojo.h
@@ -87,21 +87,21 @@
static scoped_ptr<ChannelFactory> CreateClientFactory(
const ChannelHandle& channel_handle);
- ~ChannelMojo() override;
+ virtual ~ChannelMojo();
// ChannelMojoHost tells the client handle using this API.
void OnClientLaunched(base::ProcessHandle handle);
// Channel implementation
- bool Connect() override;
- void Close() override;
- bool Send(Message* message) override;
- base::ProcessId GetPeerPID() const override;
- base::ProcessId GetSelfPID() const override;
+ virtual bool Connect() override;
+ virtual void Close() override;
+ virtual bool Send(Message* message) override;
+ virtual base::ProcessId GetPeerPID() const override;
+ virtual base::ProcessId GetSelfPID() const override;
#if defined(OS_POSIX) && !defined(OS_NACL)
- int GetClientFileDescriptor() const override;
- base::ScopedFD TakeClientFileDescriptor() override;
+ virtual int GetClientFileDescriptor() const override;
+ virtual base::ScopedFD TakeClientFileDescriptor() override;
// These access protected API of IPC::Message, which has ChannelMojo
// as a friend class.
@@ -114,8 +114,9 @@
#endif // defined(OS_POSIX) && !defined(OS_NACL)
// MojoBootstrapDelegate implementation
- void OnPipeAvailable(mojo::embedder::ScopedPlatformHandle handle) override;
- void OnBootstrapError() override;
+ virtual void OnPipeAvailable(
+ mojo::embedder::ScopedPlatformHandle handle) override;
+ virtual void OnBootstrapError() override;
// Called from MessagePipeReader implementations
void OnMessageReceived(Message& message);
diff --git a/ipc/mojo/ipc_channel_mojo_host.cc b/ipc/mojo/ipc_channel_mojo_host.cc
index beb18ee..6f9075a 100644
--- a/ipc/mojo/ipc_channel_mojo_host.cc
+++ b/ipc/mojo/ipc_channel_mojo_host.cc
@@ -17,12 +17,12 @@
class ChannelMojoHost::ChannelDelegate : public ChannelMojo::Delegate {
public:
explicit ChannelDelegate(scoped_refptr<base::TaskRunner> io_task_runner);
- ~ChannelDelegate() override;
+ virtual ~ChannelDelegate();
// ChannelMojo::Delegate
- base::WeakPtr<Delegate> ToWeakPtr() override;
- void OnChannelCreated(base::WeakPtr<ChannelMojo> channel) override;
- scoped_refptr<base::TaskRunner> GetIOTaskRunner() override;
+ virtual base::WeakPtr<Delegate> ToWeakPtr() override;
+ virtual void OnChannelCreated(base::WeakPtr<ChannelMojo> channel) override;
+ virtual scoped_refptr<base::TaskRunner> GetIOTaskRunner() override;
// Returns an weak ptr of ChannelDelegate instead of Delegate
base::WeakPtr<ChannelDelegate> GetWeakPtr();
diff --git a/ipc/mojo/ipc_channel_mojo_readers.h b/ipc/mojo/ipc_channel_mojo_readers.h
index cadf453..fae62ad 100644
--- a/ipc/mojo/ipc_channel_mojo_readers.h
+++ b/ipc/mojo/ipc_channel_mojo_readers.h
@@ -33,9 +33,9 @@
bool Send(scoped_ptr<Message> message);
// MessagePipeReader implementation
- void OnMessageReceived() override;
- void OnPipeClosed() override;
- void OnPipeError(MojoResult error) override;
+ virtual void OnMessageReceived() override;
+ virtual void OnPipeClosed() override;
+ virtual void OnPipeError(MojoResult error) override;
private:
ChannelMojo* owner_;
@@ -52,8 +52,8 @@
virtual bool Connect();
// MessagePipeReader implementation
- void OnPipeClosed() override;
- void OnPipeError(MojoResult error) override;
+ virtual void OnPipeClosed() override;
+ virtual void OnPipeError(MojoResult error) override;
protected:
ChannelMojo* owner_;
@@ -65,13 +65,13 @@
class ServerControlReader : public ControlReader {
public:
ServerControlReader(mojo::ScopedMessagePipeHandle pipe, ChannelMojo* owner);
- ~ServerControlReader() override;
+ virtual ~ServerControlReader();
// ControlReader override
- bool Connect() override;
+ virtual bool Connect() override;
// MessagePipeReader implementation
- void OnMessageReceived() override;
+ virtual void OnMessageReceived() override;
private:
MojoResult SendHelloRequest();
@@ -88,7 +88,7 @@
ClientControlReader(mojo::ScopedMessagePipeHandle pipe, ChannelMojo* owner);
// MessagePipeReader implementation
- void OnMessageReceived() override;
+ virtual void OnMessageReceived() override;
private:
MojoResult RespondHelloRequest(MojoHandle message_channel);
diff --git a/ipc/mojo/ipc_channel_mojo_unittest.cc b/ipc/mojo/ipc_channel_mojo_unittest.cc
index 1697ff7..cefa29e 100644
--- a/ipc/mojo/ipc_channel_mojo_unittest.cc
+++ b/ipc/mojo/ipc_channel_mojo_unittest.cc
@@ -27,9 +27,9 @@
ListenerThatExpectsOK()
: received_ok_(false) {}
- ~ListenerThatExpectsOK() override {}
+ virtual ~ListenerThatExpectsOK() {}
- bool OnMessageReceived(const IPC::Message& message) override {
+ virtual bool OnMessageReceived(const IPC::Message& message) override {
PickleIterator iter(message);
std::string should_be_ok;
EXPECT_TRUE(iter.ReadString(&should_be_ok));
@@ -39,7 +39,7 @@
return true;
}
- void OnChannelError() override {
+ virtual void OnChannelError() override {
// The connection should be healthy while the listener is waiting
// message. An error can occur after that because the peer
// process dies.
@@ -79,7 +79,7 @@
class IPCChannelMojoTest : public IPCTestBase {
protected:
- scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
+ virtual scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
const IPC::ChannelHandle& handle,
base::TaskRunner* runner) override {
host_.reset(new IPC::ChannelMojoHost(task_runner()));
@@ -87,7 +87,7 @@
handle);
}
- bool DidStartClient() override {
+ virtual bool DidStartClient() override {
bool ok = IPCTestBase::DidStartClient();
DCHECK(ok);
host_->OnClientLaunched(client_process());
@@ -106,7 +106,7 @@
: is_connected_called_(false) {
}
- void OnChannelConnected(int32 peer_pid) override {
+ virtual void OnChannelConnected(int32 peer_pid) override {
IPC::TestChannelListener::OnChannelConnected(peer_pid);
EXPECT_TRUE(base::kNullProcessId != peer_pid);
is_connected_called_ = true;
@@ -165,13 +165,15 @@
: has_error_(false) {
}
- void OnChannelConnected(int32 peer_pid) override {
+ virtual void OnChannelConnected(int32 peer_pid) override {
base::MessageLoop::current()->Quit();
}
- bool OnMessageReceived(const IPC::Message& message) override { return true; }
+ virtual bool OnMessageReceived(const IPC::Message& message) override {
+ return true;
+ }
- void OnChannelError() override {
+ virtual void OnChannelError() override {
has_error_ = true;
base::MessageLoop::current()->Quit();
}
@@ -185,7 +187,7 @@
class IPCChannelMojoErrorTest : public IPCTestBase {
protected:
- scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
+ virtual scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
const IPC::ChannelHandle& handle,
base::TaskRunner* runner) override {
host_.reset(new IPC::ChannelMojoHost(task_runner()));
@@ -193,7 +195,7 @@
handle);
}
- bool DidStartClient() override {
+ virtual bool DidStartClient() override {
bool ok = IPCTestBase::DidStartClient();
DCHECK(ok);
host_->OnClientLaunched(client_process());
@@ -209,9 +211,11 @@
ListenerThatQuits() {
}
- bool OnMessageReceived(const IPC::Message& message) override { return true; }
+ virtual bool OnMessageReceived(const IPC::Message& message) override {
+ return true;
+ }
- void OnChannelConnected(int32 peer_pid) override {
+ virtual void OnChannelConnected(int32 peer_pid) override {
base::MessageLoop::current()->Quit();
}
};
@@ -320,9 +324,9 @@
ListenerThatExpectsFile()
: sender_(NULL) {}
- ~ListenerThatExpectsFile() override {}
+ virtual ~ListenerThatExpectsFile() {}
- bool OnMessageReceived(const IPC::Message& message) override {
+ virtual bool OnMessageReceived(const IPC::Message& message) override {
PickleIterator iter(message);
base::ScopedFD fd;
@@ -336,7 +340,9 @@
return true;
}
- void OnChannelError() override { NOTREACHED(); }
+ virtual void OnChannelError() override {
+ NOTREACHED();
+ }
static std::string GetSendingFileContent() {
return "Hello";
diff --git a/ipc/mojo/ipc_mojo_bootstrap.cc b/ipc/mojo/ipc_mojo_bootstrap.cc
index 7458265..7233de0 100644
--- a/ipc/mojo/ipc_mojo_bootstrap.cc
+++ b/ipc/mojo/ipc_mojo_bootstrap.cc
@@ -20,15 +20,15 @@
public:
MojoServerBootstrap();
- void OnClientLaunched(base::ProcessHandle process) override;
+ virtual void OnClientLaunched(base::ProcessHandle process) override;
private:
void SendClientPipe();
void SendClientPipeIfReady();
// Listener implementations
- bool OnMessageReceived(const Message& message) override;
- void OnChannelConnected(int32 peer_pid) override;
+ virtual bool OnMessageReceived(const Message& message) override;
+ virtual void OnChannelConnected(int32 peer_pid) override;
mojo::embedder::ScopedPlatformHandle server_pipe_;
base::ProcessHandle client_process_;
@@ -112,12 +112,12 @@
public:
MojoClientBootstrap();
- void OnClientLaunched(base::ProcessHandle process) override;
+ virtual void OnClientLaunched(base::ProcessHandle process) override;
private:
// Listener implementations
- bool OnMessageReceived(const Message& message) override;
- void OnChannelConnected(int32 peer_pid) override;
+ virtual bool OnMessageReceived(const Message& message) override;
+ virtual void OnChannelConnected(int32 peer_pid) override;
DISALLOW_COPY_AND_ASSIGN(MojoClientBootstrap);
};
diff --git a/ipc/mojo/ipc_mojo_bootstrap.h b/ipc/mojo/ipc_mojo_bootstrap.h
index 0d007dd..540becf 100644
--- a/ipc/mojo/ipc_mojo_bootstrap.h
+++ b/ipc/mojo/ipc_mojo_bootstrap.h
@@ -41,7 +41,7 @@
Delegate* delegate);
MojoBootstrap();
- ~MojoBootstrap() override;
+ virtual ~MojoBootstrap();
// Start the handshake over the underlying platform channel.
bool Connect();
@@ -68,8 +68,8 @@
void Init(scoped_ptr<Channel> channel, Delegate* delegate);
// Listener implementations
- void OnBadMessageReceived(const Message& message) override;
- void OnChannelError() override;
+ virtual void OnBadMessageReceived(const Message& message) override;
+ virtual void OnChannelError() override;
scoped_ptr<Channel> channel_;
Delegate* delegate_;
diff --git a/ipc/mojo/ipc_mojo_bootstrap_unittest.cc b/ipc/mojo/ipc_mojo_bootstrap_unittest.cc
index a071d66..8bffd88 100644
--- a/ipc/mojo/ipc_mojo_bootstrap_unittest.cc
+++ b/ipc/mojo/ipc_mojo_bootstrap_unittest.cc
@@ -23,8 +23,9 @@
public:
TestingDelegate() : passed_(false) {}
- void OnPipeAvailable(mojo::embedder::ScopedPlatformHandle handle) override;
- void OnBootstrapError() override;
+ virtual void OnPipeAvailable(
+ mojo::embedder::ScopedPlatformHandle handle) override;
+ virtual void OnBootstrapError() override;
bool passed() const { return passed_; }
diff --git a/ipc/mojo/ipc_mojo_perftest.cc b/ipc/mojo/ipc_mojo_perftest.cc
index 29e7839..dca405e 100644
--- a/ipc/mojo/ipc_mojo_perftest.cc
+++ b/ipc/mojo/ipc_mojo_perftest.cc
@@ -29,7 +29,7 @@
MojoChannelPerfTest();
- scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
+ virtual scoped_ptr<IPC::ChannelFactory> CreateChannelFactory(
const IPC::ChannelHandle& handle,
base::TaskRunner* runner) override {
host_.reset(new IPC::ChannelMojoHost(task_runner()));
@@ -37,7 +37,7 @@
handle);
}
- bool DidStartClient() override {
+ virtual bool DidStartClient() override {
bool ok = IPCTestBase::DidStartClient();
DCHECK(ok);
host_->OnClientLaunched(client_process());
@@ -73,7 +73,8 @@
MojoTestClient();
- scoped_ptr<IPC::Channel> CreateChannel(IPC::Listener* listener) override;
+ virtual scoped_ptr<IPC::Channel> CreateChannel(
+ IPC::Listener* listener) override;
};
MojoTestClient::MojoTestClient() {