Mojo: More Channel tests.

mojo/system/channel.cc

 // Copyright 2013 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 "mojo/system/channel.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 74 matching lines @@
        it != to_destroy.end();
        ++it) {
     if (it->second.state == EndpointInfo::STATE_NORMAL) {
       it->second.message_pipe->OnRemove(it->second.port);
       num_live++;
     } else {
       DCHECK(!it->second.message_pipe.get());
       num_zombies++;
     }
   }
-  DVLOG(2) << "Shut down Channel with " << num_live << " live endpoints and "
-           << num_zombies << " zombies";
+  DVLOG_IF(2, num_live || num_zombies)
+      << "Shut down Channel with " << num_live << " live endpoints and "
+      << num_zombies << " zombies";
 }
 
 MessageInTransit::EndpointId Channel::AttachMessagePipeEndpoint(
     scoped_refptr<MessagePipe> message_pipe,
     unsigned port) {
   DCHECK(message_pipe);
   DCHECK(port == 0 || port == 1);
 
   MessageInTransit::EndpointId local_id;
   {
@@ skipping 366 matching lines @@
   // TODO(vtl): Is this how we really want to handle this?
   // Sometimes we'll want to propagate the error back to the message pipe
   // (endpoint), and notify it that the remote is (effectively) closed.
   // Sometimes we'll want to kill the channel (and notify all the endpoints that
   // their remotes are dead.
   LOG(WARNING) << error_message;
 }
 
 }  // namespace system
 }  // namespace mojo