Kill bad apps.

extensions/browser/guest_view/app_view/app_view_guest.h

 // 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.
 
 #ifndef EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_
 #define EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_
 
 #include "base/id_map.h"
 #include "components/guest_view/browser/guest_view.h"
 #include "extensions/browser/guest_view/app_view/app_view_guest_delegate.h"
 
 namespace extensions {
 class Extension;
 class ExtensionHost;
 
 // An AppViewGuest provides the browser-side implementation of <appview> API.
 // AppViewGuest is created on attachment. That is, when a guest WebContents is
 // associated with a particular embedder WebContents. This happens on calls to
 // the connect API.
 class AppViewGuest : public guest_view::GuestView<AppViewGuest> {
  public:
   static const char Type[];
 
   // Completes the creation of a WebContents associated with the provided
-  // |guest_extensions_id|.
+  // |guest_extensions_id|. |guest_render_process_host| is the RenderProcessHost
+  // for the guest contents calling this method.

[lfg, 2015/07/06 18:46:21]

I know it's not part of your modifications, but could you also document the
parameters that aren't documented?

[EhsanK, 2015/07/06 21:09:59]

Sure. Done.

   static bool CompletePendingRequest(
       content::BrowserContext* browser_context,
       const GURL& url,
       int guest_instance_id,
-      const std::string& guest_extension_id);
+      const std::string& guest_extension_id,
+      content::RenderProcessHost* guest_render_process_host);
 
   static GuestViewBase* Create(content::WebContents* owner_web_contents);
 
+  static std::vector<int> GetAllRegisteredInstanceIdsForTesting();
+
   // content::WebContentsDelegate implementation.
   bool HandleContextMenu(const content::ContextMenuParams& params) override;
   void RequestMediaAccessPermission(
       content::WebContents* web_contents,
       const content::MediaStreamRequest& request,
       const content::MediaResponseCallback& callback) override;
   bool CheckMediaAccessPermission(content::WebContents* web_contents,
                                   const GURL& security_origin,
                                   content::MediaStreamType type) override;
 
@@ skipping 29 matching lines @@
   // This is used to ensure pending tasks will not fire after this object is
   // destroyed.
   base::WeakPtrFactory<AppViewGuest> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AppViewGuest);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_GUEST_VIEW_APP_VIEW_APP_VIEW_GUEST_H_