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

Unified Diff: blimp/net/blimp_connection.cc

Issue 2019463002: Reland: Added a debug info UI for Blimp (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed an ownership issue Created 4 years, 7 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/blimp_connection.cc
diff --git a/blimp/net/blimp_connection.cc b/blimp/net/blimp_connection.cc
index 9e8cd6ae7c8216da9d64d86764c3bd98739251b3..7b79a5f3f9dc229258372d0fee0bea972600a23f 100644
--- a/blimp/net/blimp_connection.cc
+++ b/blimp/net/blimp_connection.cc
@@ -20,7 +20,6 @@
#include "net/base/completion_callback.h"
namespace blimp {
-namespace {
// Forwards incoming blimp messages to PacketWriter.
class BlimpMessageSender : public BlimpMessageProcessor {
@@ -106,8 +105,6 @@ void BlimpMessageSender::OnWritePacketComplete(int result) {
process_callback.Run(result);
}
-} // namespace
-
BlimpConnection::BlimpConnection(std::unique_ptr<PacketReader> reader,
std::unique_ptr<PacketWriter> writer)
: reader_(std::move(reader)),
@@ -118,9 +115,7 @@ BlimpConnection::BlimpConnection(std::unique_ptr<PacketReader> reader,
DCHECK(reader_);
message_pump_->set_error_observer(this);
- BlimpMessageSender* sender =
- static_cast<BlimpMessageSender*>(outgoing_msg_processor_.get());
- sender->set_error_observer(this);
+ outgoing_msg_processor_->set_error_observer(this);
}
BlimpConnection::BlimpConnection() {}

Powered by Google App Engine
This is Rietveld 408576698