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

Side by Side Diff: services/js/echo_apptest.cc

Issue 1916233002: Mark ApplicationImpl::ConnectTo{Application,Service}() as deprecated. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: 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 unified diff | Download patch
« no previous file with comments | « services/java_handler/java_handler.cc ('k') | services/js/network_apptest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "services/js/test/echo_service.mojom.h" 5 #include "services/js/test/echo_service.mojom.h"
6 #include "services/js/test/js_application_test_base.h" 6 #include "services/js/test/js_application_test_base.h"
7 7
8 using mojo::String; 8 using mojo::String;
9 9
10 namespace js { 10 namespace js {
11 namespace { 11 namespace {
12 12
13 class JSEchoTest : public test::JSApplicationTestBase { 13 class JSEchoTest : public test::JSApplicationTestBase {
14 public: 14 public:
15 JSEchoTest() : JSApplicationTestBase() {} 15 JSEchoTest() : JSApplicationTestBase() {}
16 ~JSEchoTest() override {} 16 ~JSEchoTest() override {}
17 17
18 protected: 18 protected:
19 // ApplicationTestBase: 19 // ApplicationTestBase:
20 void SetUp() override { 20 void SetUp() override {
21 ApplicationTestBase::SetUp(); 21 ApplicationTestBase::SetUp();
22 const std::string& url = JSAppURL("echo.js"); 22 const std::string& url = JSAppURL("echo.js");
23 application_impl()->ConnectToService(url, &echo_service_); 23 application_impl()->ConnectToServiceDeprecated(url, &echo_service_);
24 } 24 }
25 25
26 mojo::EchoServicePtr echo_service_; 26 mojo::EchoServicePtr echo_service_;
27 27
28 private: 28 private:
29 MOJO_DISALLOW_COPY_AND_ASSIGN(JSEchoTest); 29 MOJO_DISALLOW_COPY_AND_ASSIGN(JSEchoTest);
30 }; 30 };
31 31
32 class JSServiceProviderEchoTest : public test::JSApplicationTestBase { 32 class JSServiceProviderEchoTest : public test::JSApplicationTestBase {
33 public: 33 public:
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 String foo; 115 String foo;
116 EchoStringCallback callback(&foo); 116 EchoStringCallback callback(&foo);
117 echo_service->EchoString("foo", callback); 117 echo_service->EchoString("foo", callback);
118 EXPECT_TRUE(echo_service.WaitForIncomingResponse()); 118 EXPECT_TRUE(echo_service.WaitForIncomingResponse());
119 EXPECT_EQ("foo", foo); 119 EXPECT_EQ("foo", foo);
120 echo_service->Quit(); 120 echo_service->Quit();
121 } 121 }
122 122
123 } // namespace 123 } // namespace
124 } // namespace js 124 } // namespace js
OLDNEW
« no previous file with comments | « services/java_handler/java_handler.cc ('k') | services/js/network_apptest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698