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

Unified Diff: net/dns/mojo_host_resolver_impl_unittest.cc

Issue 2589663003: mojo:: Rename mojo::GetProxy() to mojo::MakeRequest() (Closed)
Patch Set: Rebase Created 4 years 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 | « net/dns/host_resolver_mojo.cc ('k') | net/proxy/in_process_mojo_proxy_resolver_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/dns/mojo_host_resolver_impl_unittest.cc
diff --git a/net/dns/mojo_host_resolver_impl_unittest.cc b/net/dns/mojo_host_resolver_impl_unittest.cc
index 9405c71afb16d8c33d102aad9fdd443f38fc3c31..186a738927da445bbd5e71c4ca7e477bbb8bd4c2 100644
--- a/net/dns/mojo_host_resolver_impl_unittest.cc
+++ b/net/dns/mojo_host_resolver_impl_unittest.cc
@@ -162,7 +162,7 @@ class MojoHostResolverImplTest : public testing::Test {
TEST_F(MojoHostResolverImplTest, Resolve) {
interfaces::HostResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
resolver_service_->Resolve(CreateRequest("example.com", 80, false),
std::move(client_ptr));
@@ -176,7 +176,7 @@ TEST_F(MojoHostResolverImplTest, Resolve) {
TEST_F(MojoHostResolverImplTest, ResolveSynchronous) {
interfaces::HostResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
mock_host_resolver_.set_synchronous_mode(true);
@@ -192,9 +192,9 @@ TEST_F(MojoHostResolverImplTest, ResolveSynchronous) {
TEST_F(MojoHostResolverImplTest, ResolveMultiple) {
interfaces::HostResolverRequestClientPtr client1_ptr;
- TestRequestClient client1(mojo::GetProxy(&client1_ptr));
+ TestRequestClient client1(mojo::MakeRequest(&client1_ptr));
interfaces::HostResolverRequestClientPtr client2_ptr;
- TestRequestClient client2(mojo::GetProxy(&client2_ptr));
+ TestRequestClient client2(mojo::MakeRequest(&client2_ptr));
mock_host_resolver_.set_ondemand_mode(true);
@@ -220,9 +220,9 @@ TEST_F(MojoHostResolverImplTest, ResolveMultiple) {
TEST_F(MojoHostResolverImplTest, ResolveDuplicate) {
interfaces::HostResolverRequestClientPtr client1_ptr;
- TestRequestClient client1(mojo::GetProxy(&client1_ptr));
+ TestRequestClient client1(mojo::MakeRequest(&client1_ptr));
interfaces::HostResolverRequestClientPtr client2_ptr;
- TestRequestClient client2(mojo::GetProxy(&client2_ptr));
+ TestRequestClient client2(mojo::MakeRequest(&client2_ptr));
mock_host_resolver_.set_ondemand_mode(true);
@@ -248,7 +248,7 @@ TEST_F(MojoHostResolverImplTest, ResolveDuplicate) {
TEST_F(MojoHostResolverImplTest, ResolveFailure) {
interfaces::HostResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
resolver_service_->Resolve(CreateRequest("failure.fail", 80, false),
std::move(client_ptr));
@@ -261,7 +261,7 @@ TEST_F(MojoHostResolverImplTest, ResolveFailure) {
TEST_F(MojoHostResolverImplTest, DestroyClient) {
interfaces::HostResolverRequestClientPtr client_ptr;
std::unique_ptr<TestRequestClient> client(
- new TestRequestClient(mojo::GetProxy(&client_ptr)));
+ new TestRequestClient(mojo::MakeRequest(&client_ptr)));
mock_host_resolver_.set_ondemand_mode(true);
« no previous file with comments | « net/dns/host_resolver_mojo.cc ('k') | net/proxy/in_process_mojo_proxy_resolver_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698