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

Unified Diff: mojo/public/cpp/application/application_impl_base.h

Issue 1985223003: Factor stuff from ApplicationImpl out to a new class, ApplicationImplBase. (Closed) Base URL: https://github.com/domokit/mojo.git@work790_app_test_base_no_app_impl
Patch Set: fix android Created 4 years, 7 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: mojo/public/cpp/application/application_impl_base.h
diff --git a/mojo/public/cpp/application/application_impl_base.h b/mojo/public/cpp/application/application_impl_base.h
new file mode 100644
index 0000000000000000000000000000000000000000..04109d7eefe71e09da09ed52914e449944e859fd
--- /dev/null
+++ b/mojo/public/cpp/application/application_impl_base.h
@@ -0,0 +1,98 @@
+// Copyright 2016 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_PUBLIC_CPP_APPLICATION_APPLICATION_IMPL_BASE_H_
+#define MOJO_PUBLIC_CPP_APPLICATION_APPLICATION_IMPL_BASE_H_
+
+#include <memory>
+#include <string>
+#include <vector>
+
+#include "mojo/public/cpp/bindings/binding.h"
+#include "mojo/public/cpp/bindings/interface_request.h"
+#include "mojo/public/cpp/system/macros.h"
+#include "mojo/public/interfaces/application/application.mojom.h"
+#include "mojo/public/interfaces/application/shell.mojom.h"
+
+namespace mojo {
+
+class ServiceProviderImpl;
+
+// Base helper class for implementing the |Application| interface, which the
+// shell uses for basic communication with an application (e.g., to connect
+// clients to services provided by an application).
+//
+// To use this class, subclass it and implement/override the required methods
+// (see below).
+//
+// TODO(vtl): ApplicationRunners should take this instead of an
+// ApplicationDelegate. Write more here when that's true (it's pretty hard to
+// use this class in the current setup).
+class ApplicationImplBase : public Application {
+ public:
+ explicit ApplicationImplBase(
+ InterfaceRequest<Application> application_request);
+ ~ApplicationImplBase() override;
+
+ // Quits the main run loop for this application.
+ // TODO(vtl): This is implemented in application_runner.cc (for example). Its
+ // presence here is pretty dubious.
+ static void Terminate();
+
+ // This will be valid after |Initialize()| has been received and remain valid
+ // until this object is destroyed.
+ Shell* shell() const { return shell_.get(); }
+
+ // Returns any initial configuration arguments, passed by the shell.
+ const std::vector<std::string>& args() const { return args_; }
+ bool HasArg(const std::string& arg) const;
+
+ const std::string& url() const { return url_; }
+
+ // Methods to be implemented/overridden by subclasses:
+
+ // Called after |Initialize()| has been received (|shell()|, |args()|, and
+ // |url()| will be valid when this is called. The default implementation does
+ // nothing.
+ virtual void OnInitialize() {}
+
+ // Called when another application connects to this application (i.e., we
+ // receive |AcceptConnection()|). This should either configure what services
+ // are "provided" (made available via a |ServiceProvider|) to that application
+ // and return true, or this may return false to reject the connection
+ // entirely.
+ virtual bool OnAcceptConnection(
+ ServiceProviderImpl* service_provider_impl) = 0;
+
+ // Called before quitting the main message (run) loop, i.e., before
+ // |Terminate()|. The default implementation does nothing.
+ virtual void OnQuit() {}
+
+ private:
+ // |Application| implementation. In general, you probably shouldn't call these
+ // directly (but I can't really stop you).
+ void Initialize(InterfaceHandle<Shell> shell,
+ Array<String> args,
+ const mojo::String& url) final;
+ void AcceptConnection(const String& requestor_url,
+ InterfaceRequest<ServiceProvider> services,
+ InterfaceHandle<ServiceProvider> exposed_services,
+ const String& url) final;
+ void RequestQuit() final;
+
+ Binding<Application> application_binding_;
+
+ // Set by |Initialize()|.
+ ShellPtr shell_;
+ std::vector<std::string> args_;
+ std::string url_;
+
+ std::vector<std::unique_ptr<ServiceProviderImpl>> service_provider_impls_;
+
+ MOJO_DISALLOW_COPY_AND_ASSIGN(ApplicationImplBase);
+};
+
+} // namespace mojo
+
+#endif // MOJO_PUBLIC_CPP_APPLICATION_APPLICATION_IMPL_BASE_H_

Powered by Google App Engine
This is Rietveld 408576698