Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3706)

Unified Diff: google_apis/gcm/engine/connection_handler_impl.h

Issue 649283003: Standardize usage of virtual/override/final in google_apis/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: google_apis/gcm/engine/connection_handler_impl.h
diff --git a/google_apis/gcm/engine/connection_handler_impl.h b/google_apis/gcm/engine/connection_handler_impl.h
index 296bf6dbf86ba15befa40c36d7e04b34a897a7fc..fc965fb7c99d0c2022b341772374241e2130df0c 100644
--- a/google_apis/gcm/engine/connection_handler_impl.h
+++ b/google_apis/gcm/engine/connection_handler_impl.h
@@ -30,15 +30,14 @@ class GCM_EXPORT ConnectionHandlerImpl : public ConnectionHandler {
const ProtoReceivedCallback& read_callback,
const ProtoSentCallback& write_callback,
const ConnectionChangedCallback& connection_callback);
- virtual ~ConnectionHandlerImpl();
+ ~ConnectionHandlerImpl() override;
// ConnectionHandler implementation.
- virtual void Init(const mcs_proto::LoginRequest& login_request,
- net::StreamSocket* socket) override;
- virtual void Reset() override;
- virtual bool CanSendMessage() const override;
- virtual void SendMessage(const google::protobuf::MessageLite& message)
- override;
+ void Init(const mcs_proto::LoginRequest& login_request,
+ net::StreamSocket* socket) override;
+ void Reset() override;
+ bool CanSendMessage() const override;
+ void SendMessage(const google::protobuf::MessageLite& message) override;
private:
// State machine for handling incoming data. See WaitForData(..) for usage.
« no previous file with comments | « google_apis/gcm/engine/connection_factory_impl_unittest.cc ('k') | google_apis/gcm/engine/fake_connection_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698