[Extensions] Clean up the handling of ExtensionHostMsg_Request

extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc

Index: extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
diff --git a/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
index 61fd14276bb5dc7fcdd404b985c47deaa86e0c22..2f4466236e9f673befc1f418598b18a76fd64fcf 100644
--- a/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
+++ b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
@@ -82,14 +82,6 @@ MimeHandlerViewGuest::MimeHandlerViewGuest(
 MimeHandlerViewGuest::~MimeHandlerViewGuest() {
 }
 
-WindowController* MimeHandlerViewGuest::GetExtensionWindowController() const {
-  return nullptr;
-}
-
-WebContents* MimeHandlerViewGuest::GetAssociatedWebContents() const {
-  return web_contents();
-}
-
 const char* MimeHandlerViewGuest::GetAPINamespace() const {
   return "mimeHandlerViewGuestInternal";
 }
@@ -155,10 +147,7 @@ void MimeHandlerViewGuest::DidAttachToEmbedder() {
 
 void MimeHandlerViewGuest::DidInitialize(
     const base::DictionaryValue& create_params) {
-  extension_function_dispatcher_.reset(
-      new ExtensionFunctionDispatcher(browser_context(), this));
-  if (delegate_)
-    delegate_->AttachHelpers();
+  ExtensionsAPIClient::Get()->AttachWebContentsHelpers(web_contents());

[not at google - send to devlin, 2015/06/10 18:37:49]

It seems like this belongs in the GuestView base class, since guest view relies
on extension bindings to work. In fact many of these GuestView::DidInitialize
implementations *only* call AttachWebContentsHelpers(), so you could remove
those.

I'll let fsamuel@ comment on this, though.

[Devlin, 2015/06/10 20:59:20]

The only concern is that it would introduce a dependency from
components/guest_view to extensions, or would require wiring up (another)
delegate to this.  If we decide to do it (which I'd be fine with - it would be a
nice little clean up), I'd probably want to push it to a separate patch, if
that's okay (as is, this will still clean up quite a bit of code and introduces
no new dependencies).

 }
 
 bool MimeHandlerViewGuest::ZoomPropagatesFromEmbedderToGuest() const {
@@ -251,27 +240,10 @@ void MimeHandlerViewGuest::DocumentOnLoadCompletedInMainFrame() {
           element_instance_id()));
 }
 
-bool MimeHandlerViewGuest::OnMessageReceived(const IPC::Message& message) {
-  bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP(MimeHandlerViewGuest, message)
-    IPC_MESSAGE_HANDLER(ExtensionHostMsg_Request, OnRequest)
-    IPC_MESSAGE_UNHANDLED(handled = false)
-  IPC_END_MESSAGE_MAP()
-  return handled;
-}
-
 base::WeakPtr<StreamContainer> MimeHandlerViewGuest::GetStream() const {
   if (!stream_)
     return base::WeakPtr<StreamContainer>();
   return stream_->GetWeakPtr();
 }
 
-void MimeHandlerViewGuest::OnRequest(
-    const ExtensionHostMsg_Request_Params& params) {
-  if (extension_function_dispatcher_) {
-    extension_function_dispatcher_->Dispatch(
-        params, web_contents()->GetRenderViewHost());
-  }
-}
-
 }  // namespace extensions