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

Unified Diff: net/http/http_network_transaction.cc

Issue 10243016: CHECK for that we are not seeing user-visible SPDY_PING_FAILED errors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index 7b527666f233f9694b7b242b56537f4b42ded427..d11aebbfa016ef3dac7d4b897a8e498d3f215156 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -164,6 +164,7 @@ int HttpNetworkTransaction::Start(const HttpRequestInfo* request_info,
int rv = DoLoop(OK);
if (rv == ERR_IO_PENDING)
callback_ = callback;
+ CHECK_NE(rv, ERR_SPDY_PING_FAILED);
return rv;
}
@@ -470,6 +471,7 @@ bool HttpNetworkTransaction::is_https_request() const {
}
void HttpNetworkTransaction::DoCallback(int rv) {
+ CHECK_NE(rv, ERR_SPDY_PING_FAILED);
DCHECK_NE(rv, ERR_IO_PENDING);
DCHECK(!callback_.is_null());
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698