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

Unified Diff: net/proxy/mojo_proxy_resolver_factory_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/proxy/in_process_mojo_proxy_resolver_factory.cc ('k') | net/proxy/mojo_proxy_resolver_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc
diff --git a/net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc b/net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc
index a266131f7bda479b38d5f82db3ac803f84b28d22..945e9e55a01a8e9ce0c806aeeaa8dfc7b9149473 100644
--- a/net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc
+++ b/net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc
@@ -101,7 +101,7 @@ class MojoProxyResolverFactoryImplTest
mock_factory_ = new TestProxyResolverFactory(&waiter_);
mojo::MakeStrongBinding(base::MakeUnique<MojoProxyResolverFactoryImpl>(
base::WrapUnique(mock_factory_)),
- mojo::GetProxy(&factory_));
+ mojo::MakeRequest(&factory_));
}
void OnConnectionError() { waiter_.NotifyEvent(CONNECTION_ERROR); }
@@ -135,8 +135,8 @@ TEST_F(MojoProxyResolverFactoryImplTest, DisconnectProxyResolverClient) {
interfaces::ProxyResolverPtr proxy_resolver;
interfaces::ProxyResolverFactoryRequestClientPtr client_ptr;
mojo::Binding<ProxyResolverFactoryRequestClient> client_binding(
- this, mojo::GetProxy(&client_ptr));
- factory_->CreateResolver(kScriptData, mojo::GetProxy(&proxy_resolver),
+ this, mojo::MakeRequest(&client_ptr));
+ factory_->CreateResolver(kScriptData, mojo::MakeRequest(&proxy_resolver),
std::move(client_ptr));
proxy_resolver.set_connection_error_handler(
base::Bind(&MojoProxyResolverFactoryImplTest::OnConnectionError,
@@ -162,8 +162,8 @@ TEST_F(MojoProxyResolverFactoryImplTest, Error) {
interfaces::ProxyResolverPtr proxy_resolver;
interfaces::ProxyResolverFactoryRequestClientPtr client_ptr;
mojo::Binding<ProxyResolverFactoryRequestClient> client_binding(
- this, mojo::GetProxy(&client_ptr));
- factory_->CreateResolver(kScriptData, mojo::GetProxy(&proxy_resolver),
+ this, mojo::MakeRequest(&client_ptr));
+ factory_->CreateResolver(kScriptData, mojo::MakeRequest(&proxy_resolver),
std::move(client_ptr));
proxy_resolver.set_connection_error_handler(
base::Bind(&MojoProxyResolverFactoryImplTest::OnConnectionError,
@@ -183,8 +183,8 @@ TEST_F(MojoProxyResolverFactoryImplTest,
interfaces::ProxyResolverPtr proxy_resolver;
interfaces::ProxyResolverFactoryRequestClientPtr client_ptr;
mojo::Binding<ProxyResolverFactoryRequestClient> client_binding(
- this, mojo::GetProxy(&client_ptr));
- factory_->CreateResolver(kScriptData, mojo::GetProxy(&proxy_resolver),
+ this, mojo::MakeRequest(&client_ptr));
+ factory_->CreateResolver(kScriptData, mojo::MakeRequest(&proxy_resolver),
std::move(client_ptr));
proxy_resolver.set_connection_error_handler(
base::Bind(&MojoProxyResolverFactoryImplTest::OnConnectionError,
@@ -201,8 +201,8 @@ TEST_F(MojoProxyResolverFactoryImplTest,
interfaces::ProxyResolverPtr proxy_resolver;
interfaces::ProxyResolverFactoryRequestClientPtr client_ptr;
mojo::Binding<ProxyResolverFactoryRequestClient> client_binding(
- this, mojo::GetProxy(&client_ptr));
- factory_->CreateResolver(kScriptData, mojo::GetProxy(&proxy_resolver),
+ this, mojo::MakeRequest(&client_ptr));
+ factory_->CreateResolver(kScriptData, mojo::MakeRequest(&proxy_resolver),
std::move(client_ptr));
proxy_resolver.set_connection_error_handler(
base::Bind(&MojoProxyResolverFactoryImplTest::OnConnectionError,
« no previous file with comments | « net/proxy/in_process_mojo_proxy_resolver_factory.cc ('k') | net/proxy/mojo_proxy_resolver_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698