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

Unified Diff: mojo/shell/package_test_package.cc

Issue 1706833003: Move various Shell tests into a subdir. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@conn
Patch Set: . Created 4 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 | « mojo/shell/package_test_app_b_manifest.json ('k') | mojo/shell/package_test_package_manifest.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/package_test_package.cc
diff --git a/mojo/shell/package_test_package.cc b/mojo/shell/package_test_package.cc
deleted file mode 100644
index 53a8bdf881973c49885901cf312353ab4fcbc57a..0000000000000000000000000000000000000000
--- a/mojo/shell/package_test_package.cc
+++ /dev/null
@@ -1,166 +0,0 @@
-// 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.
-
-#include <stddef.h>
-#include <stdint.h>
-
-#include <utility>
-
-#include "base/bind.h"
-#include "base/macros.h"
-#include "base/run_loop.h"
-#include "base/threading/simple_thread.h"
-#include "mojo/public/c/system/main.h"
-#include "mojo/public/cpp/bindings/weak_binding_set.h"
-#include "mojo/shell/package_test.mojom.h"
-#include "mojo/shell/public/cpp/application_runner.h"
-#include "mojo/shell/public/cpp/interface_factory.h"
-#include "mojo/shell/public/cpp/shell.h"
-#include "mojo/shell/public/cpp/shell_client.h"
-#include "mojo/shell/public/interfaces/shell_client_factory.mojom.h"
-
-// Tests that multiple applications can be packaged in a single Mojo application
-// implementing ShellClientFactory; that these applications can be specified by
-// the package's manifest and are thus registered with the PackageManager.
-
-namespace mojo {
-namespace shell {
-
-using GetNameCallback = test::mojom::PackageTestService::GetNameCallback;
-
-class ProvidedShellClient
- : public ShellClient,
- public InterfaceFactory<test::mojom::PackageTestService>,
- public test::mojom::PackageTestService,
- public base::SimpleThread {
- public:
- ProvidedShellClient(const std::string& name,
- mojom::ShellClientRequest request)
- : base::SimpleThread(name),
- name_(name),
- request_(std::move(request)),
- shell_(nullptr) {
- Start();
- }
- ~ProvidedShellClient() override {
- Join();
- }
-
- private:
- // mojo::ShellClient:
- void Initialize(Shell* shell, const std::string& url, uint32_t id) override {
- shell_ = shell;
- bindings_.set_connection_error_handler(
- base::Bind(&ProvidedShellClient::OnConnectionError,
- base::Unretained(this)));
- }
- bool AcceptConnection(Connection* connection) override {
- connection->AddInterface<test::mojom::PackageTestService>(
- this);
- return true;
- }
-
- // InterfaceFactory<test::mojom::PackageTestService>:
- void Create(Connection* connection,
- test::mojom::PackageTestServiceRequest request) override {
- bindings_.AddBinding(this, std::move(request));
- }
-
- // test::mojom::PackageTestService:
- void GetName(const GetNameCallback& callback) override {
- callback.Run(name_);
- }
-
- // base::SimpleThread:
- void Run() override {
- ApplicationRunner(this).Run(request_.PassMessagePipe().release().value(),
- false);
- delete this;
- }
-
- void OnConnectionError() {
- if (bindings_.empty())
- shell_->Quit();
- }
-
- const std::string name_;
- mojom::ShellClientRequest request_;
- Shell* shell_;
- WeakBindingSet<test::mojom::PackageTestService> bindings_;
-
- DISALLOW_COPY_AND_ASSIGN(ProvidedShellClient);
-};
-
-class PackageTestShellClient
- : public ShellClient,
- public InterfaceFactory<mojom::ShellClientFactory>,
- public InterfaceFactory<test::mojom::PackageTestService>,
- public mojom::ShellClientFactory,
- public test::mojom::PackageTestService {
- public:
- PackageTestShellClient() : shell_(nullptr) {}
- ~PackageTestShellClient() override {}
-
- private:
- // mojo::ShellClient:
- void Initialize(Shell* shell, const std::string& url, uint32_t id) override {
- shell_ = shell;
- bindings_.set_connection_error_handler(
- base::Bind(&PackageTestShellClient::OnConnectionError,
- base::Unretained(this)));
- }
- bool AcceptConnection(Connection* connection) override {
- connection->AddInterface<ShellClientFactory>(this);
- connection->AddInterface<test::mojom::PackageTestService>(this);
- return true;
- }
-
- // InterfaceFactory<mojom::ShellClientFactory>:
- void Create(Connection* connection,
- mojom::ShellClientFactoryRequest request) override {
- shell_client_factory_bindings_.AddBinding(this, std::move(request));
- }
-
- // InterfaceFactory<test::mojom::PackageTestService>:
- void Create(Connection* connection,
- test::mojom::PackageTestServiceRequest request) override {
- bindings_.AddBinding(this, std::move(request));
- }
-
- // mojom::ShellClientFactory:
- void CreateShellClient(mojom::ShellClientRequest request,
- const String& url) override {
- if (url == "mojo://package_test_a/")
- new ProvidedShellClient("A", std::move(request));
- else if (url == "mojo://package_test_b/")
- new ProvidedShellClient("B", std::move(request));
- }
-
- // test::mojom::PackageTestService:
- void GetName(const GetNameCallback& callback) override {
- callback.Run("ROOT");
- }
-
- void OnConnectionError() {
- if (bindings_.empty())
- shell_->Quit();
- }
-
- Shell* shell_;
- std::vector<scoped_ptr<ShellClient>> delegates_;
- WeakBindingSet<mojom::ShellClientFactory> shell_client_factory_bindings_;
- WeakBindingSet<test::mojom::PackageTestService> bindings_;
-
- DISALLOW_COPY_AND_ASSIGN(PackageTestShellClient);
-};
-
-} // namespace shell
-} // namespace mojo
-
-
-MojoResult MojoMain(MojoHandle shell_handle) {
- MojoResult rv = mojo::ApplicationRunner(
- new mojo::shell::PackageTestShellClient).Run(shell_handle);
- return rv;
-}
« no previous file with comments | « mojo/shell/package_test_app_b_manifest.json ('k') | mojo/shell/package_test_package_manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698