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

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

Issue 669303002: Add an extensions keep-alive client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@keep-alive-service
Patch Set: keep alive client tests 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 | « extensions/renderer/dispatcher.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
new file mode 100644
index 0000000000000000000000000000000000000000..2e76dd35aedb8a9624629028f416654189778f68
--- /dev/null
+++ b/extensions/renderer/mojo/keep_alive_client_unittest.cc
@@ -0,0 +1,94 @@
+// 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 "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/interface_impl.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 mojo::InterfaceImpl<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,
+ mojo::InterfaceRequest<KeepAlive> keep_alive) {
+ mojo::BindToRequest(new TestKeepAlive(on_destruction), &keep_alive);
+ on_creation.Run();
+ }
+
+ private:
+ const base::Closure on_destruction_;
+};
+
+} // namespace
+
+class KeepAliveClientTest : public ApiTestBase {
+ public:
+ KeepAliveClientTest() {}
+
+ void SetUp() override {
+ ApiTestBase::SetUp();
+ service_provider()->AddService(
+ 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/dispatcher.cc ('k') | extensions/renderer/resources/binding.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698