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

Unified Diff: blimp/net/browser_connection_handler.cc

Issue 1985863002: Incorporate BlobChannel into Blimp image encode/decode pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blobchannel-helium
Patch Set: moved the blob serialization processor to the compositor manager Created 4 years, 6 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
Index: blimp/net/browser_connection_handler.cc
diff --git a/blimp/net/browser_connection_handler.cc b/blimp/net/browser_connection_handler.cc
index 37519f7b9db40ca747886ab306e11324ba40367b..00599535c69f5cfa16504214f115ee86dcbdbb11 100644
--- a/blimp/net/browser_connection_handler.cc
+++ b/blimp/net/browser_connection_handler.cc
@@ -63,7 +63,6 @@ void BrowserConnectionHandler::OnConnectionError(int error) {
}
void BrowserConnectionHandler::DropCurrentConnection() {
- DCHECK(connection_);
output_buffer_->SetOutputProcessor(nullptr);
Wez 2016/06/10 22:29:54 nit: You might consider adding an early-return in
Kevin M 2016/06/10 23:51:50 Done.
connection_.reset();
}

Powered by Google App Engine
This is Rietveld 408576698