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

Unified Diff: net/proxy/proxy_resolver_factory_mojo_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/proxy/proxy_resolver_factory_mojo.cc ('k') | services/catalog/catalog.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/proxy/proxy_resolver_factory_mojo_unittest.cc
diff --git a/net/proxy/proxy_resolver_factory_mojo_unittest.cc b/net/proxy/proxy_resolver_factory_mojo_unittest.cc
index a53bbd034e3f498872788c37868ac3a5871d1fca..1ea381fb955b6e9fbd03e2274c76a0e0324f322f 100644
--- a/net/proxy/proxy_resolver_factory_mojo_unittest.cc
+++ b/net/proxy/proxy_resolver_factory_mojo_unittest.cc
@@ -279,7 +279,7 @@ void MockMojoProxyResolver::GetProxyForUrl(
auto request = base::MakeUnique<HostResolver::RequestInfo>(
HostPortPair(url.spec(), 12345));
interfaces::HostResolverRequestClientPtr dns_client;
- mojo::GetProxy(&dns_client);
+ mojo::MakeRequest(&dns_client);
client->ResolveDns(std::move(request), std::move(dns_client));
blocked_clients_.push_back(
base::MakeUnique<interfaces::ProxyResolverRequestClientPtr>(
@@ -447,7 +447,7 @@ void MockMojoProxyResolverFactory::CreateResolver(
auto request = base::MakeUnique<HostResolver::RequestInfo>(
HostPortPair(pac_script, 12345));
interfaces::HostResolverRequestClientPtr dns_client;
- mojo::GetProxy(&dns_client);
+ mojo::MakeRequest(&dns_client);
client->ResolveDns(std::move(request), std::move(dns_client));
blocked_clients_.push_back(
base::MakeUnique<interfaces::ProxyResolverFactoryRequestClientPtr>(
@@ -523,7 +523,7 @@ class ProxyResolverFactoryMojoTest : public testing::Test,
public:
void SetUp() override {
mock_proxy_resolver_factory_.reset(new MockMojoProxyResolverFactory(
- &mock_proxy_resolver_, mojo::GetProxy(&factory_ptr_)));
+ &mock_proxy_resolver_, mojo::MakeRequest(&factory_ptr_)));
proxy_resolver_factory_mojo_.reset(new ProxyResolverFactoryMojo(
this, &host_resolver_,
base::Callback<std::unique_ptr<ProxyResolverErrorObserver>()>(),
« no previous file with comments | « net/proxy/proxy_resolver_factory_mojo.cc ('k') | services/catalog/catalog.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698