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

Unified Diff: blimp/net/blimp_message_pump.cc

Issue 1962393004: Added a debug info UI for Blimp (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed implementation to receive synchronous calls 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_message_pump.cc
diff --git a/blimp/net/blimp_message_pump.cc b/blimp/net/blimp_message_pump.cc
index cc53d1f0b0ed2f0f54f3882ac0983191c629125a..43916f95f4f3c2ac6a271e3be0f5ae65b099735c 100644
--- a/blimp/net/blimp_message_pump.cc
+++ b/blimp/net/blimp_message_pump.cc
@@ -24,6 +24,12 @@ BlimpMessagePump::BlimpMessagePump(PacketReader* reader)
BlimpMessagePump::~BlimpMessagePump() {}
+void BlimpMessagePump::SetBlimpConnectionStatistics(
+ BlimpConnectionStatistics* connection_statistics) {
+ DCHECK(reader_);
Kevin M 2016/05/24 01:02:01 Not needed: constructor enforces non-nullness, and
shaktisahu 2016/05/24 21:02:45 Removed function since we are passing in construct
+ reader_->set_blimp_connection_statistics(connection_statistics);
+}
+
void BlimpMessagePump::SetMessageProcessor(BlimpMessageProcessor* processor) {
DVLOG(1) << "SetMessageProcessor, processor=" << processor;
if (processor && !processor_) {

Powered by Google App Engine
This is Rietveld 408576698