Make IPC::Channel::Listener:OnMessageReceived have a return value indicating ...

chrome/renderer/renderer_main_unittest.cc

 // Copyright (c) 2010 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 "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/process_util.h"
 #include "base/test/multiprocess_test.h"
 #include "base/test/test_timeouts.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 71 matching lines @@
   return command_line;
 }
 
 // Listener class that kills the message loop when it connects.
 class SuicidalListener : public IPC::Channel::Listener {
  public:
   void OnChannelConnected(int32 peer_pid) {
     MessageLoop::current()->Quit();
   }
 
-  void OnMessageReceived(const IPC::Message& message) {
+  bool OnMessageReceived(const IPC::Message& message) {
     // We shouldn't receive any messages
-    ASSERT_TRUE(false);
+    NOTREACHED();
+    return false;
   }
 };
 
 MULTIPROCESS_TEST_MAIN(SimpleRenderer) {
   SandboxInitWrapper dummy_sandbox_init;
   CommandLine cl(*CommandLine::ForCurrentProcess());
   cl.AppendSwitchASCII(switches::kProcessChannelID, kRendererTestChannelName);
 
   MainFunctionParams dummy_params(cl, dummy_sandbox_init, NULL);
   return RendererMain(dummy_params);
@@ skipping 11 matching lines @@
 
   MessageLoop::current()->Run();
 
   // The renderer should exit when we close the channel.
   control_channel.Close();
 
   EXPECT_TRUE(base::WaitForSingleProcess(renderer_pid,
                                          TestTimeouts::action_timeout_ms()));
 }
 #endif  // defined(OS_POSIX)