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

Unified Diff: content/common/mojo/service_registry_impl.h

Issue 671663002: Standardize usage of virtual/override/final in content/ (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
« no previous file with comments | « content/common/message_router.h ('k') | content/common/net/url_request_user_data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/mojo/service_registry_impl.h
diff --git a/content/common/mojo/service_registry_impl.h b/content/common/mojo/service_registry_impl.h
index bdfb97603fdeec261b3a730db4e03365115fd620..b3a6576b27ac75f43059f9ab1af3bcb093938ddc 100644
--- a/content/common/mojo/service_registry_impl.h
+++ b/content/common/mojo/service_registry_impl.h
@@ -26,7 +26,7 @@ class CONTENT_EXPORT ServiceRegistryImpl
public:
ServiceRegistryImpl();
explicit ServiceRegistryImpl(mojo::ScopedMessagePipeHandle handle);
- virtual ~ServiceRegistryImpl();
+ ~ServiceRegistryImpl() override;
// Binds to a remote ServiceProvider. This will expose added services to the
// remote ServiceProvider with the corresponding handle and enable
@@ -35,23 +35,20 @@ class CONTENT_EXPORT ServiceRegistryImpl
void BindRemoteServiceProvider(mojo::ScopedMessagePipeHandle handle);
// ServiceRegistry overrides.
- virtual void AddService(
- const std::string& service_name,
- const base::Callback<void(mojo::ScopedMessagePipeHandle)> service_factory)
- override;
- virtual void RemoveService(const std::string& service_name) override;
- virtual void ConnectToRemoteService(
- const base::StringPiece& service_name,
- mojo::ScopedMessagePipeHandle handle) override;
+ void AddService(const std::string& service_name,
+ const base::Callback<void(mojo::ScopedMessagePipeHandle)>
+ service_factory) override;
+ void RemoveService(const std::string& service_name) override;
+ void ConnectToRemoteService(const base::StringPiece& service_name,
+ mojo::ScopedMessagePipeHandle handle) override;
base::WeakPtr<ServiceRegistry> GetWeakPtr();
private:
// mojo::InterfaceImpl<mojo::ServiceProvider> overrides.
- virtual void ConnectToService(
- const mojo::String& name,
- mojo::ScopedMessagePipeHandle client_handle) override;
- virtual void OnConnectionError() override;
+ void ConnectToService(const mojo::String& name,
+ mojo::ScopedMessagePipeHandle client_handle) override;
+ void OnConnectionError() override;
std::map<std::string, base::Callback<void(mojo::ScopedMessagePipeHandle)> >
service_factories_;
« no previous file with comments | « content/common/message_router.h ('k') | content/common/net/url_request_user_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698