Add delegate to install web_contents helpers for MimeHandlerView.

extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc

 // Copyright 2014 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 "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
 
 #include "base/strings/stringprintf.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/common/url_constants.h"
+#include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/guest_view/guest_view_constants.h"
 #include "extensions/browser/guest_view/guest_view_manager.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_constants.h"
+#include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest_delegate.h"
 #include "extensions/common/feature_switch.h"
 #include "extensions/strings/grit/extensions_strings.h"
 #include "net/base/url_util.h"
 
 using content::WebContents;
 
 namespace extensions {
 
 // static
 const char MimeHandlerViewGuest::Type[] = "mimehandler";
 
 // static
 GuestViewBase* MimeHandlerViewGuest::Create(
     content::BrowserContext* browser_context,
     int guest_instance_id) {
   if (!extensions::FeatureSwitch::mime_handler_view()->IsEnabled())
     return NULL;
 
   return new MimeHandlerViewGuest(browser_context, guest_instance_id);
 }
 
 MimeHandlerViewGuest::MimeHandlerViewGuest(
     content::BrowserContext* browser_context,
     int guest_instance_id)
-    : GuestView<MimeHandlerViewGuest>(browser_context, guest_instance_id) {
+    : GuestView<MimeHandlerViewGuest>(browser_context, guest_instance_id),
+      delegate_(
+          ExtensionsAPIClient::Get()->CreateMimeHandlerViewGuestDelegate()) {

[Fady Samuel, 2014/09/09 14:30:29]

Perhaps we can pass in "this" into "CreateMimeHandlerViewGuestDelegate"?

[lazyboy, 2014/09/09 15:49:33]

Done.

 }
 
 MimeHandlerViewGuest::~MimeHandlerViewGuest() {
 }
 
 const char* MimeHandlerViewGuest::GetAPINamespace() const {
   return "mimeHandlerViewGuestInternal";
 }
 
 int MimeHandlerViewGuest::GetTaskPrefix() const {
@@ skipping 38 matching lines @@
   std::string src;
   DCHECK(attach_params()->GetString(mime_handler_view::kSrc, &src) &&
          !src.empty());
   web_contents()->GetController().LoadURL(
       GURL(src),
       content::Referrer(),
       content::PAGE_TRANSITION_AUTO_TOPLEVEL,
       std::string());
 }
 
+void MimeHandlerViewGuest::DidInitialize() {
+  if (delegate_)
+    delegate_->AttachHelpers(web_contents());

[Fady Samuel, 2014/09/09 14:30:29]

If the delegate_ knows about this, then we don't need to pass in web_contents().

[lazyboy, 2014/09/09 15:49:33]

Done.

+}
+
 }  // namespace extensions