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 2c5dee1b7d490985c4152d43ed8fd24eff7e323e..997f9b219dd61ddcf59b3542d896a9134c837eb9 100644 |
--- a/net/proxy/proxy_resolver_factory_mojo_unittest.cc |
+++ b/net/proxy/proxy_resolver_factory_mojo_unittest.cc |
@@ -28,7 +28,6 @@ |
#include "net/log/net_log_with_source.h" |
#include "net/log/test_net_log.h" |
#include "net/proxy/mojo_proxy_resolver_factory.h" |
-#include "net/proxy/mojo_proxy_type_converters.h" |
#include "net/proxy/proxy_info.h" |
#include "net/proxy/proxy_resolver.h" |
#include "net/proxy/proxy_resolver_error_observer.h" |
@@ -127,12 +126,7 @@ struct GetProxyForUrlAction { |
}; |
GetProxyForUrlAction() {} |
- GetProxyForUrlAction(const GetProxyForUrlAction& old) { |
- action = old.action; |
- error = old.error; |
- expected_url = old.expected_url; |
- proxy_servers = old.proxy_servers.Clone(); |
- } |
+ GetProxyForUrlAction(const GetProxyForUrlAction& other) = default; |
static GetProxyForUrlAction ReturnError(const GURL& url, Error error) { |
GetProxyForUrlAction result; |
@@ -141,12 +135,11 @@ struct GetProxyForUrlAction { |
return result; |
} |
- static GetProxyForUrlAction ReturnServers( |
- const GURL& url, |
- const mojo::Array<interfaces::ProxyServerPtr>& proxy_servers) { |
+ static GetProxyForUrlAction ReturnServers(const GURL& url, |
+ const ProxyInfo& proxy_info) { |
GetProxyForUrlAction result; |
result.expected_url = url; |
- result.proxy_servers = proxy_servers.Clone(); |
+ result.proxy_info = proxy_info; |
return result; |
} |
@@ -180,7 +173,7 @@ struct GetProxyForUrlAction { |
Action action = COMPLETE; |
Error error = OK; |
- mojo::Array<interfaces::ProxyServerPtr> proxy_servers; |
+ ProxyInfo proxy_info; |
GURL expected_url; |
}; |
@@ -263,7 +256,7 @@ void MockMojoProxyResolver::GetProxyForUrl( |
client->OnError(12345, url.spec()); |
switch (action.action) { |
case GetProxyForUrlAction::COMPLETE: { |
- client->ReportResult(action.error, std::move(action.proxy_servers)); |
+ client->ReportResult(action.error, action.proxy_info); |
break; |
} |
case GetProxyForUrlAction::DROP: { |
@@ -284,10 +277,8 @@ void MockMojoProxyResolver::GetProxyForUrl( |
break; |
} |
case GetProxyForUrlAction::MAKE_DNS_REQUEST: { |
- interfaces::HostResolverRequestInfoPtr request( |
- interfaces::HostResolverRequestInfo::New()); |
- request->host = url.spec(); |
- request->port = 12345; |
+ auto request = base::MakeUnique<HostResolver::RequestInfo>( |
+ HostPortPair(url.spec(), 12345)); |
interfaces::HostResolverRequestClientPtr dns_client; |
mojo::GetProxy(&dns_client); |
client->ResolveDns(std::move(request), std::move(dns_client)); |
@@ -453,10 +444,8 @@ void MockMojoProxyResolverFactory::CreateResolver( |
break; |
} |
case CreateProxyResolverAction::MAKE_DNS_REQUEST: { |
- interfaces::HostResolverRequestInfoPtr request( |
- interfaces::HostResolverRequestInfo::New()); |
- request->host = pac_script; |
- request->port = 12345; |
+ auto request = base::MakeUnique<HostResolver::RequestInfo>( |
+ HostPortPair(pac_script, 12345)); |
interfaces::HostResolverRequestClientPtr dns_client; |
mojo::GetProxy(&dns_client); |
client->ResolveDns(std::move(request), std::move(dns_client)); |
@@ -554,13 +543,10 @@ class ProxyResolverFactoryMojoTest : public testing::Test, |
on_delete_callback_.closure()); |
} |
- mojo::Array<interfaces::ProxyServerPtr> ProxyServersFromPacString( |
- const std::string& pac_string) { |
+ ProxyInfo ProxyServersFromPacString(const std::string& pac_string) { |
ProxyInfo proxy_info; |
proxy_info.UsePacString(pac_string); |
- |
- return mojo::Array<interfaces::ProxyServerPtr>::From( |
- proxy_info.proxy_list().GetAll()); |
+ return proxy_info; |
} |
void CreateProxyResolver() { |