Clean up Automation and Chrome Frame IPC code.

chrome_frame/external_tab.h

 // 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.
 
 #ifndef CHROME_FRAME_EXTERNAL_TAB_H_
 #define CHROME_FRAME_EXTERNAL_TAB_H_
 #pragma once
 
 #include <windows.h>
 #include <atlbase.h>
@@ skipping 18 matching lines @@
 
 namespace IPC {
   struct NavigationInfo;
   struct MiniContextMenuParams;
 }
 
 // This is the delegate/callback interface that has to be implemented
 // by the customers of ExternalTabProxy class.
 class UIDelegate {
  public:
-  virtual void OnNavigationStateChanged(int flags,
-      const IPC::NavigationInfo& nav_info) = 0;
+  virtual void OnNavigationStateChanged(
+      int flags, const NavigationInfo& nav_info) = 0;
   virtual void OnUpdateTargetUrl(const std::wstring& new_target_url) = 0;
   virtual void OnExtensionInstalled(const FilePath& path, void* user_data,
       AutomationMsg_ExtensionResponseValues response) = 0;
   virtual void OnLoad(const GURL& url) = 0;
-  virtual void OnMessageFromChromeFrame(const std::string& message,
-      const std::string& origin, const std::string& target) = 0;
-  virtual void OnHandleContextMenu(HANDLE menu_handle, int align_flags,
-      const IPC::MiniContextMenuParams& params) = 0;
+  virtual void OnMessageFromChromeFrame(
+      const std::string& message, const std::string& origin,
+      const std::string& target) = 0;
+  virtual void OnHandleContextMenu(
+      HANDLE menu_handle, int align_flags,
+      const MiniContextMenuParams& params) = 0;
   virtual void OnHandleAccelerator(const MSG& accel_message) = 0;
   virtual void OnTabbedOut(bool reverse) = 0;
   virtual void OnGoToHistoryOffset(int offset) = 0;
-  virtual void OnOpenURL(const GURL& url_to_open, const GURL& referrer,
-                       int open_disposition) = 0;
+  virtual void OnOpenURL(
+      const GURL& url_to_open, const GURL& referrer, int open_disposition) = 0;
  protected:
   ~UIDelegate() {}
 };
 
 struct CreateTabParams {
   struct ProxyParams proxy_params;
   bool is_incognito;
   bool is_widget_mode;
   GURL url;
   GURL referrer;
@@ skipping 19 matching lines @@
                          public ChromeProxyDelegate {
  public:
   ExternalTabProxy();
   ~ExternalTabProxy();
 
 #ifdef UNIT_TEST
   void set_proxy_factory(ChromeProxyFactory* factory) {
     proxy_factory_ = factory;
   }
 #endif
+
+  // IPC::Channel::Listener implementation.
+  void OnMessageReceived(const IPC::Message& message);
+
   //
   virtual void CreateTab(const CreateTabParams& create_params,
                          UIDelegate* delegate);
   virtual void Navigate(const std::string& url, const std::string& referrer,
                         NavigationConstraints* navigation_constraints);
   virtual void NavigateToIndex(int index);
   virtual void ForwardMessageFromExternalHost(const std::string& message,
       const std::string& origin, const std::string& target);
   virtual void ChromeFrameHostMoved();
 
@@ skipping 16 matching lines @@
 
   //////////////////////////////////////////////////////////////////////////
   // ChromeProxyDelegate implementation
   virtual int tab_handle() {
     return tab_;
   }
   virtual void Connected(ChromeProxy* proxy);
   virtual void PeerLost(ChromeProxy* proxy, DisconnectReason reason);
   virtual void Disconnected();
 
-
   // Sync message responses.
   virtual void Completed_CreateTab(bool success, HWND chrome_wnd,
       HWND tab_window, int tab_handle, int session_id);
   virtual void Completed_ConnectToTab(bool success, HWND chrome_window,
       HWND tab_window, int tab_handle, int session_id);
   virtual void Completed_Navigate(bool success,
       enum AutomationMsg_NavigationResponseValues res);
   virtual void Completed_InstallExtension(bool success,
       enum AutomationMsg_ExtensionResponseValues res, SyncMessageContext* ctx);
   virtual void Completed_LoadExpandedExtension(bool success,
       enum AutomationMsg_ExtensionResponseValues res, SyncMessageContext* ctx);
   virtual void Completed_GetEnabledExtensions(bool success,
       const std::vector<FilePath>* extensions);
 
   // Network requests from Chrome.
-  virtual void Network_Start(int request_id,
-      const IPC::AutomationURLRequest& request_info);
-  virtual void Network_Read(int request_id, int bytes_to_read);
-  virtual void Network_End(int request_id, const URLRequestStatus& s);
-  virtual void Network_DownloadInHost(int request_id);
-  virtual void GetCookies(const GURL& url, int cookie_id);
-  virtual void SetCookie(const GURL& url, const std::string& cookie);
+  virtual void OnNetwork_Start(
+      int request_id, const AutomationURLRequest& request_info);
+  virtual void OnNetwork_Read(int request_id, int bytes_to_read);
+  virtual void OnNetwork_End(int request_id, const URLRequestStatus& s);
+  virtual void OnNetwork_DownloadInHost(int request_id);
+  virtual void OnGetCookies(const GURL& url, int cookie_id);
+  virtual void OnSetCookie(const GURL& url, const std::string& cookie);
 
   // Navigation progress notifications.
-  virtual void NavigationStateChanged(int flags,
-      const IPC::NavigationInfo& nav_info);
-  virtual void UpdateTargetUrl(const std::wstring& url);
-  virtual void NavigationFailed(int error_code, const GURL& gurl);
-  virtual void DidNavigate(const IPC::NavigationInfo& navigation_info);
-  virtual void TabLoaded(const GURL& url);
+  virtual void OnNavigationStateChanged(
+      int flags, const NavigationInfo& nav_info);
+  virtual void OnUpdateTargetUrl(const std::wstring& url);
+  virtual void OnNavigationFailed(int error_code, const GURL& gurl);
+  virtual void OnDidNavigate(const NavigationInfo& navigation_info);
+  virtual void OnTabLoaded(const GURL& url);
 
-  virtual void OpenURL(const GURL& url_to_open, const GURL& referrer,
-                       int open_disposition);
-  virtual void GoToHistoryOffset(int offset);
-  virtual void MessageToHost(const std::string& message,
-      const std::string& origin, const std::string& target);
+  virtual void OnOpenURL(const GURL& url_to_open, const GURL& referrer,
+                         int open_disposition);
+  virtual void OnGoToHistoryOffset(int offset);
+  virtual void OnMessageToHost(
+      const std::string& message, const std::string& origin,
+      const std::string& target);
 
   // Misc. UI.
-  virtual void HandleAccelerator(const MSG& accel_message);
-  virtual void HandleContextMenu(HANDLE menu_handle, int align_flags,
-                                 const IPC::MiniContextMenuParams& params);
-  virtual void TabbedOut(bool reverse);
+  virtual void OnHandleAccelerator(const MSG& accel_message);
+  virtual void OnHandleContextMenu(HANDLE menu_handle, int align_flags,
+                                   const MiniContextMenuParams& params);
+  virtual void OnTabbedOut(bool reverse);
 
   // Other
-  virtual void TabClosed();
-  virtual void AttachTab(const IPC::AttachExternalTabParams& attach_params);
+  virtual void OnTabClosed();
+  virtual void OnAttachTab(const AttachExternalTabParams& attach_params);
 
   // end of ChromeProxyDelegate methods
   //////////////////////////////////////////////////////////////////////////
   void Init();
   void Destroy();
 
   // The UiXXXX are the ChromeProxyDelegate methods but on UI thread.
   void UiConnected(ChromeProxy* proxy);
   void UiPeerLost(ChromeProxy* proxy, DisconnectReason reason);
   void UiCompleted_CreateTab(bool success, HWND chrome_window,
@@ skipping 30 matching lines @@
     GURL url;
     GURL referrer;
     void Set(const GURL& gurl, const GURL& ref) {
       url = gurl;
       referrer = ref;
     }
   } pending_navigation_;
 };
 
 #endif  // CHROME_FRAME_EXTERNAL_TAB_H_