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

Unified Diff: net/http/http_stream_factory_impl_job_controller.cc

Issue 2347163002: s/NPN/ALPN/ (Closed)
Patch Set: Re: #12. Created 4 years, 3 months 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/http/http_stream_factory_impl_job_controller.h ('k') | net/http/http_stream_factory_impl_request.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl_job_controller.cc
diff --git a/net/http/http_stream_factory_impl_job_controller.cc b/net/http/http_stream_factory_impl_job_controller.cc
index 75c28e0d8271ba5e77f3411f77017dd4a67e0c45..17784a1d2b44356cf7a9d67274f138075daeb24b 100644
--- a/net/http/http_stream_factory_impl_job_controller.cc
+++ b/net/http/http_stream_factory_impl_job_controller.cc
@@ -175,7 +175,7 @@ void HttpStreamFactoryImpl::JobController::OnStreamReady(
std::unique_ptr<HttpStream> stream = job->ReleaseStream();
DCHECK(stream);
- MarkRequestComplete(job->was_npn_negotiated(), job->negotiated_protocol(),
+ MarkRequestComplete(job->was_alpn_negotiated(), job->negotiated_protocol(),
job->using_spdy());
if (!request_)
@@ -198,7 +198,7 @@ void HttpStreamFactoryImpl::JobController::OnBidirectionalStreamImplReady(
return;
}
- MarkRequestComplete(job->was_npn_negotiated(), job->negotiated_protocol(),
+ MarkRequestComplete(job->was_alpn_negotiated(), job->negotiated_protocol(),
job->using_spdy());
if (!request_)
@@ -220,7 +220,7 @@ void HttpStreamFactoryImpl::JobController::OnWebSocketHandshakeStreamReady(
const ProxyInfo& used_proxy_info,
WebSocketHandshakeStreamBase* stream) {
DCHECK(job);
- MarkRequestComplete(job->was_npn_negotiated(), job->negotiated_protocol(),
+ MarkRequestComplete(job->was_alpn_negotiated(), job->negotiated_protocol(),
job->using_spdy());
if (!request_)
@@ -410,7 +410,7 @@ void HttpStreamFactoryImpl::JobController::OnNewSpdySessionReady(
// Cache these values in case the job gets deleted.
const SSLConfig used_ssl_config = job->server_ssl_config();
const ProxyInfo used_proxy_info = job->proxy_info();
- const bool was_npn_negotiated = job->was_npn_negotiated();
+ const bool was_alpn_negotiated = job->was_alpn_negotiated();
const NextProto negotiated_protocol = job->negotiated_protocol();
const bool using_spdy = job->using_spdy();
const BoundNetLog net_log = job->net_log();
@@ -430,7 +430,7 @@ void HttpStreamFactoryImpl::JobController::OnNewSpdySessionReady(
BindJob(job);
}
- MarkRequestComplete(was_npn_negotiated, negotiated_protocol, using_spdy);
+ MarkRequestComplete(was_alpn_negotiated, negotiated_protocol, using_spdy);
std::unique_ptr<HttpStream> stream;
std::unique_ptr<BidirectionalStreamImpl> bidirectional_stream_impl;
@@ -456,7 +456,7 @@ void HttpStreamFactoryImpl::JobController::OnNewSpdySessionReady(
// Notify |factory_|. |request_| and |bounded_job_| might be deleted already.
if (spdy_session && spdy_session->IsAvailable()) {
factory->OnNewSpdySessionReady(spdy_session, direct, used_ssl_config,
- used_proxy_info, was_npn_negotiated,
+ used_proxy_info, was_alpn_negotiated,
negotiated_protocol, using_spdy, net_log);
}
if (is_job_orphaned) {
@@ -749,11 +749,11 @@ void HttpStreamFactoryImpl::JobController::OnJobSucceeded(Job* job) {
}
void HttpStreamFactoryImpl::JobController::MarkRequestComplete(
- bool was_npn_negotiated,
+ bool was_alpn_negotiated,
NextProto negotiated_protocol,
bool using_spdy) {
if (request_)
- request_->Complete(was_npn_negotiated, negotiated_protocol, using_spdy);
+ request_->Complete(was_alpn_negotiated, negotiated_protocol, using_spdy);
}
void HttpStreamFactoryImpl::JobController::OnAlternativeJobFailed(Job* job) {
« no previous file with comments | « net/http/http_stream_factory_impl_job_controller.h ('k') | net/http/http_stream_factory_impl_request.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698