Rename negotiated protocol methods and members.
It's crazy that ProxyClientSocket and its three subclasses have a method called
GetProtocolNegotiated() and another one called GetNegotiatedProtocol(), with the
same signature, but very different semantics. This CL fixes that.
In src/net/, there is
8 occurrences of GetProtocolNegotiated();
55 occurrences of GetNegotiatedProtocol();
25 occurrences of protocol_negotiated;
28 occurrences of negotiated_protocol.
So GetNegotiatedProtocol() and |negotiated_protocol| win. Therefore this CL
does the following substitutions:
s/GetProtocolNegotiated/GetProxyNegotiatedProtocol/
s/protocol_negotiated/negotiated_protocol/
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
Review-Url: https://codereview.chromium.org/2229693003
Cr-Commit-Position: refs/heads/master@{#411515}
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index d53779d..ada7a37 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -458,7 +458,7 @@
proxy_info_ = used_proxy_info;
response_.was_npn_negotiated = stream_request_->was_npn_negotiated();
response_.npn_negotiated_protocol = SSLClientSocket::NextProtoToString(
- stream_request_->protocol_negotiated());
+ stream_request_->negotiated_protocol());
response_.was_fetched_via_spdy = stream_request_->using_spdy();
response_.was_fetched_via_proxy = !proxy_info_.is_direct();
if (response_.was_fetched_via_proxy && !proxy_info_.is_empty())
