Use ConnectionErrorObserver, not callbacks, for error handling.

blimp/net/blimp_message_output_buffer.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "blimp/net/blimp_message_output_buffer.h"
 
 #include <algorithm>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
@@ skipping 126 matching lines @@
   DVLOG(3) << "Writing message (id="
            << write_buffer_.front()->message->message_id()
            << ", type=" << message_to_write->type() << ")";
 
   output_processor_->ProcessMessage(std::move(message_to_write),
                                     write_complete_cb_.callback());
   DVLOG(3) << "Queue size: " << write_buffer_.size();
 }
 
 void BlimpMessageOutputBuffer::OnWriteComplete(int result) {
-  DCHECK_LE(result, net::OK);
-  VLOG(2) << "Write complete, result=" << result;
+  VLOG(2) << "Write complete";
 

[haibinlu, 2016/05/03 21:23:05]

DCHECK_EQ(net::OK, result)

[Wez, 2016/05/07 22:13:47]

See my comment on the earlier review ;)

[Kevin M, 2016/05/24 20:47:04]

I added a DCHECK for positive values.

[Kevin M, 2016/05/24 20:47:04]

Sorry, was that in reference to the callback signature? I see that I didn't
address that point. I take "result" because CompletionCallbacks are part of the
BlimpMessageProcessor convention.

I also restored the check for result == net::OK

-  if (result == net::OK) {
-    ack_buffer_.push_back(std::move(write_buffer_.front()));
-    write_buffer_.pop_front();
-    WriteNextMessageIfReady();
-  } else {
-    // An error occurred while writing to the network connection.
-    // Stop writing more messages until a new connection is established.
-    DLOG(WARNING) << "Write error (result=" << result << ")";
-  }
+  ack_buffer_.push_back(std::move(write_buffer_.front()));
+  write_buffer_.pop_front();
+  WriteNextMessageIfReady();
 }
 
 }  // namespace blimp