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

Unified Diff: sky/viewer/content_handler_impl.h

Issue 934403002: Remove last uses of InterfaceImpl<> in Sky C++ code (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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 | « sky/tools/tester/tester.cc ('k') | sky/viewer/content_handler_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/viewer/content_handler_impl.h
diff --git a/sky/viewer/content_handler_impl.h b/sky/viewer/content_handler_impl.h
index 8e6f55224c6476d7d991761975dda1104f24776b..d5caf0c75591564a7af07bf0780c50db034f56f4 100644
--- a/sky/viewer/content_handler_impl.h
+++ b/sky/viewer/content_handler_impl.h
@@ -6,15 +6,17 @@
#define SKY_VIEWER_CONTENT_HANDLER_IMPL_H_
#include "base/message_loop/message_loop.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
#include "mojo/public/interfaces/application/shell.mojom.h"
#include "mojo/services/content_handler/public/interfaces/content_handler.mojom.h"
namespace sky {
class DocumentView;
-class ContentHandlerImpl : public mojo::InterfaceImpl<mojo::ContentHandler> {
+class ContentHandlerImpl : public mojo::ContentHandler {
public:
- ContentHandlerImpl();
+ explicit ContentHandlerImpl(
+ mojo::InterfaceRequest<mojo::ContentHandler> request);
~ContentHandlerImpl() override;
private:
@@ -22,6 +24,7 @@ class ContentHandlerImpl : public mojo::InterfaceImpl<mojo::ContentHandler> {
void StartApplication(mojo::InterfaceRequest<mojo::Application> application,
mojo::URLResponsePtr response) override;
+ mojo::StrongBinding<mojo::ContentHandler> binding_;
DISALLOW_COPY_AND_ASSIGN(ContentHandlerImpl);
};
« no previous file with comments | « sky/tools/tester/tester.cc ('k') | sky/viewer/content_handler_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698