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

Unified Diff: net/proxy/mojo_proxy_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/proxy/mojo_proxy_resolver_factory_impl_unittest.cc ('k') | net/proxy/proxy_resolver_factory_mojo.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/proxy/mojo_proxy_resolver_impl_unittest.cc
diff --git a/net/proxy/mojo_proxy_resolver_impl_unittest.cc b/net/proxy/mojo_proxy_resolver_impl_unittest.cc
index 0022478283f1cd093646f8b0e4a8af65a3aa81d7..3e32c2b142e0f96a7be5a8a9abc1a6caa2768820 100644
--- a/net/proxy/mojo_proxy_resolver_impl_unittest.cc
+++ b/net/proxy/mojo_proxy_resolver_impl_unittest.cc
@@ -208,7 +208,7 @@ class MojoProxyResolverImplTest : public testing::Test {
TEST_F(MojoProxyResolverImplTest, GetProxyForUrl) {
interfaces::ProxyResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
resolver_->GetProxyForUrl(GURL(GURL("http://example.com")),
std::move(client_ptr));
@@ -255,7 +255,7 @@ TEST_F(MojoProxyResolverImplTest, GetProxyForUrl) {
TEST_F(MojoProxyResolverImplTest, GetProxyForUrlFailure) {
interfaces::ProxyResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
resolver_->GetProxyForUrl(GURL("http://example.com"), std::move(client_ptr));
ASSERT_EQ(1u, mock_proxy_resolver_->pending_jobs().size());
@@ -273,9 +273,9 @@ TEST_F(MojoProxyResolverImplTest, GetProxyForUrlFailure) {
TEST_F(MojoProxyResolverImplTest, GetProxyForUrlMultiple) {
interfaces::ProxyResolverRequestClientPtr client_ptr1;
- TestRequestClient client1(mojo::GetProxy(&client_ptr1));
+ TestRequestClient client1(mojo::MakeRequest(&client_ptr1));
interfaces::ProxyResolverRequestClientPtr client_ptr2;
- TestRequestClient client2(mojo::GetProxy(&client_ptr2));
+ TestRequestClient client2(mojo::MakeRequest(&client_ptr2));
resolver_->GetProxyForUrl(GURL("http://example.com"), std::move(client_ptr1));
resolver_->GetProxyForUrl(GURL("https://example.com"),
@@ -316,7 +316,7 @@ TEST_F(MojoProxyResolverImplTest, GetProxyForUrlMultiple) {
TEST_F(MojoProxyResolverImplTest, DestroyClient) {
interfaces::ProxyResolverRequestClientPtr client_ptr;
std::unique_ptr<TestRequestClient> client(
- new TestRequestClient(mojo::GetProxy(&client_ptr)));
+ new TestRequestClient(mojo::MakeRequest(&client_ptr)));
resolver_->GetProxyForUrl(GURL("http://example.com"), std::move(client_ptr));
ASSERT_EQ(1u, mock_proxy_resolver_->pending_jobs().size());
@@ -330,7 +330,7 @@ TEST_F(MojoProxyResolverImplTest, DestroyClient) {
TEST_F(MojoProxyResolverImplTest, DestroyService) {
interfaces::ProxyResolverRequestClientPtr client_ptr;
- TestRequestClient client(mojo::GetProxy(&client_ptr));
+ TestRequestClient client(mojo::MakeRequest(&client_ptr));
resolver_->GetProxyForUrl(GURL("http://example.com"), std::move(client_ptr));
ASSERT_EQ(1u, mock_proxy_resolver_->pending_jobs().size());
« no previous file with comments | « net/proxy/mojo_proxy_resolver_factory_impl_unittest.cc ('k') | net/proxy/proxy_resolver_factory_mojo.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698