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

chrome_frame/external_tab.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 "chrome_frame/external_tab.h"
 #include "base/lazy_instance.h"
 #include "base/tracked.h"
 #include "base/task.h"
 #include "base/waitable_event.h"
 #include "chrome/common/automation_messages.h"
@@ skipping 16 matching lines @@
 
 ExternalTabProxy::ExternalTabProxy() : state_(NONE), tab_(0), tab_wnd_(NULL),
     chrome_wnd_(NULL), proxy_factory_(g_proxy_factory.Pointer()), proxy_(NULL),
     ui_delegate_(NULL) {
 }
 
 ExternalTabProxy::~ExternalTabProxy() {
   Destroy();
 }
 
-void ExternalTabProxy::OnMessageReceived(const IPC::Message& message) {
+bool ExternalTabProxy::OnMessageReceived(const IPC::Message& message) {
+  bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(ExternalTabProxy, message)
     IPC_MESSAGE_HANDLER(AutomationMsg_NavigationStateChanged,
                         OnNavigationStateChanged)
     IPC_MESSAGE_HANDLER(AutomationMsg_UpdateTargetUrl, OnUpdateTargetUrl)
     IPC_MESSAGE_HANDLER(AutomationMsg_HandleAccelerator, OnHandleAccelerator)
     IPC_MESSAGE_HANDLER(AutomationMsg_TabbedOut, OnTabbedOut)
     IPC_MESSAGE_HANDLER(AutomationMsg_OpenURL, OnOpenURL)
     IPC_MESSAGE_HANDLER(AutomationMsg_NavigationFailed, OnNavigationFailed)
     IPC_MESSAGE_HANDLER(AutomationMsg_DidNavigate, OnDidNavigate)
     IPC_MESSAGE_HANDLER(AutomationMsg_TabLoaded, OnTabLoaded)
     IPC_MESSAGE_HANDLER(AutomationMsg_ForwardMessageToExternalHost,
                         OnMessageToHost)
     IPC_MESSAGE_HANDLER(AutomationMsg_ForwardContextMenuToExternalHost,
                         OnHandleContextMenu)
     IPC_MESSAGE_HANDLER(AutomationMsg_RequestStart, OnNetwork_Start)
     IPC_MESSAGE_HANDLER(AutomationMsg_RequestRead, OnNetwork_Read)
     IPC_MESSAGE_HANDLER(AutomationMsg_RequestEnd, OnNetwork_End)
     IPC_MESSAGE_HANDLER(AutomationMsg_DownloadRequestInHost,
                         OnNetwork_DownloadInHost)
     IPC_MESSAGE_HANDLER(AutomationMsg_GetCookiesFromHost, OnGetCookies)
     IPC_MESSAGE_HANDLER(AutomationMsg_SetCookieAsync, OnSetCookie)
     IPC_MESSAGE_HANDLER(AutomationMsg_AttachExternalTab, OnAttachTab)
     IPC_MESSAGE_HANDLER(AutomationMsg_RequestGoToHistoryEntryOffset,
                         OnGoToHistoryOffset)
     IPC_MESSAGE_HANDLER(AutomationMsg_CloseExternalTab, OnTabClosed)
+    IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
+  return handled;
 }
 
 void ExternalTabProxy::Init() {
   if (m_hWnd == NULL) {
     // Create a window on the UI thread for marshaling messages back and forth
     // from the IPC thread. This window cannot be a message only window as the
     // external chrome tab window initially is created as a child of this window
     CWindowImpl<ExternalTabProxy>::Create(GetDesktopWindow(), NULL, NULL,
         WS_CHILDWINDOW | WS_CLIPCHILDREN | WS_CLIPSIBLINGS, WS_EX_TOOLWINDOW);
     DCHECK(m_hWnd != NULL);
@@ skipping 293 matching lines @@
 }
 
 void ExternalTabProxy::OnTabClosed() {
   // TODO(stoyan):
 }
 
 void ExternalTabProxy::OnAttachTab(
     const AttachExternalTabParams& attach_params) {
   // TODO(stoyan):
 }