Use ConnectionErrorObserver, not callbacks, for error handling.

blimp/net/blimp_connection_unittest.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_connection.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 114 matching lines @@
   net::TestCompletionCallback complete_cb_1;
   sender->ProcessMessage(CreateInputMessage(),
                          complete_cb_1.callback());
   base::ResetAndReturn(&write_packet_cb).Run(net::OK);
   EXPECT_EQ(net::OK, complete_cb_1.WaitForResult());
 
   net::TestCompletionCallback complete_cb_2;
   sender->ProcessMessage(CreateControlMessage(),
                          complete_cb_2.callback());
   base::ResetAndReturn(&write_packet_cb).Run(net::ERR_FAILED);
-  EXPECT_EQ(net::ERR_FAILED, complete_cb_2.WaitForResult());

[Wez, 2016/04/21 22:44:29]

Can we assert that the second callback has not been called, here?

[Kevin M, 2016/04/22 22:10:41]

Done.

 }
 
 }  // namespace
 }  // namespace blimp