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

Unified Diff: extensions/renderer/mojo/keep_alive_client_unittest.cc

Issue 2409073002: Use mojo from the PDF extension instead of using an extension API.
Patch Set: Created 4 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 | « extensions/renderer/module_system.cc ('k') | extensions/renderer/resources/binding.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/renderer/mojo/keep_alive_client_unittest.cc
diff --git a/extensions/renderer/mojo/keep_alive_client_unittest.cc b/extensions/renderer/mojo/keep_alive_client_unittest.cc
deleted file mode 100644
index cfdcac40d175878b37ff92913717058c29c10998..0000000000000000000000000000000000000000
--- a/extensions/renderer/mojo/keep_alive_client_unittest.cc
+++ /dev/null
@@ -1,98 +0,0 @@
-// Copyright 2014 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 <utility>
-
-#include "base/macros.h"
-#include "extensions/common/mojo/keep_alive.mojom.h"
-#include "extensions/renderer/api_test_base.h"
-#include "grit/extensions_renderer_resources.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
-
-// A test launcher for tests for the stash client defined in
-// extensions/test/data/keep_alive_client_unittest.js.
-
-namespace extensions {
-namespace {
-
-// A KeepAlive implementation that calls provided callbacks on creation and
-// destruction.
-class TestKeepAlive : public KeepAlive {
- public:
- explicit TestKeepAlive(const base::Closure& on_destruction)
- : on_destruction_(on_destruction) {}
-
- ~TestKeepAlive() override { on_destruction_.Run(); }
-
- static void Create(const base::Closure& on_creation,
- const base::Closure& on_destruction,
- KeepAliveRequest keep_alive) {
- mojo::MakeStrongBinding(base::MakeUnique<TestKeepAlive>(on_destruction),
- std::move(keep_alive));
- on_creation.Run();
- }
-
- private:
- const base::Closure on_destruction_;
-};
-
-} // namespace
-
-class KeepAliveClientTest : public ApiTestBase {
- public:
- KeepAliveClientTest() {}
-
- void SetUp() override {
- ApiTestBase::SetUp();
- interface_provider()->AddInterface(
- base::Bind(&TestKeepAlive::Create,
- base::Bind(&KeepAliveClientTest::KeepAliveCreated,
- base::Unretained(this)),
- base::Bind(&KeepAliveClientTest::KeepAliveDestroyed,
- base::Unretained(this))));
- created_keep_alive_ = false;
- destroyed_keep_alive_ = false;
- }
-
- void WaitForKeepAlive() {
- // Wait for a keep-alive to be created and destroyed.
- while (!created_keep_alive_ || !destroyed_keep_alive_) {
- base::RunLoop run_loop;
- stop_run_loop_ = run_loop.QuitClosure();
- run_loop.Run();
- }
- EXPECT_TRUE(created_keep_alive_);
- EXPECT_TRUE(destroyed_keep_alive_);
- }
-
- private:
- void KeepAliveCreated() {
- created_keep_alive_ = true;
- if (!stop_run_loop_.is_null())
- stop_run_loop_.Run();
- }
- void KeepAliveDestroyed() {
- destroyed_keep_alive_ = true;
- if (!stop_run_loop_.is_null())
- stop_run_loop_.Run();
- }
-
- bool created_keep_alive_;
- bool destroyed_keep_alive_;
- base::Closure stop_run_loop_;
-
- DISALLOW_COPY_AND_ASSIGN(KeepAliveClientTest);
-};
-
-TEST_F(KeepAliveClientTest, KeepAliveWithSuccessfulCall) {
- RunTest("keep_alive_client_unittest.js", "testKeepAliveWithSuccessfulCall");
- WaitForKeepAlive();
-}
-
-TEST_F(KeepAliveClientTest, KeepAliveWithError) {
- RunTest("keep_alive_client_unittest.js", "testKeepAliveWithError");
- WaitForKeepAlive();
-}
-
-} // namespace extensions
« no previous file with comments | « extensions/renderer/module_system.cc ('k') | extensions/renderer/resources/binding.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698