Kill bad apps.

extensions/browser/guest_view/app_view/app_view_guest.h

Index: extensions/browser/guest_view/app_view/app_view_guest.h
diff --git a/extensions/browser/guest_view/app_view/app_view_guest.h b/extensions/browser/guest_view/app_view/app_view_guest.h
index 4b05d1adb2be8ccbe6c1d5151f0ed24172106d9d..b1ca13cdef44eed95b770bb80151d15bf5f3ac34 100644
--- a/extensions/browser/guest_view/app_view/app_view_guest.h
+++ b/extensions/browser/guest_view/app_view/app_view_guest.h
@@ -25,14 +25,20 @@ class AppViewGuest : public guest_view::GuestView<AppViewGuest>,
 
   // Completes the creation of a WebContents associated with the provided
   // |guest_extensions_id|.
-  static bool CompletePendingRequest(
-      content::BrowserContext* browser_context,
-      const GURL& url,
-      int guest_instance_id,
-      const std::string& guest_extension_id);
+  static bool CompletePendingRequest(content::BrowserContext* browser_context,
+                                     const GURL& url,
+                                     int guest_instance_id,
+                                     const std::string& guest_extension_id,
+                                     int guest_render_process_host_id);

[lfg, 2015/06/12 21:43:47]

Update the comment above to reflect the new parameters.

[EhsanK, 2015/06/25 16:19:51]

Done.

[lfg, 2015/06/26 21:46:05]

Forgot to update this?

[EhsanK, 2015/06/30 16:53:10]

Yes. :(. Done.

 
   static GuestViewBase* Create(content::WebContents* owner_web_contents);
 
+  static void AddFakeRequestInfoForTesting(
+      const Extension* guest_extension,
+      AppViewGuest* app_view_guest,
+      GuestViewBase::WebContentsCreatedCallback& callback,
+      int guest_instance_id);

[lfg, 2015/06/12 21:43:47]

Why did you make this a static and pass the app_view_guest as a parameter? Any
reason for not making it a member function?

[EhsanK, 2015/06/25 16:19:51]

None except that this function is nothing but an "insert-into-map" method where
the map is also static. I will change it to member function if that pattern is
preferred.

[lfg, 2015/06/26 21:46:05]

I just thought it was strange to be a static and pass the object as a
parameters. I'm not sure what's the best style here, I'll defer to fsamuel@ on
this one.

[EhsanK, 2015/06/30 16:53:10]

I ended up going towards another direction. I have removed the test for the case
where upon calling "CompletePendingRequest" the <appview> is "nullptr". For
other cases, it turns out the javascript solution works.

+
   // ExtensionFunctionDispatcher::Delegate implementation.
   WindowController* GetExtensionWindowController() const override;
   content::WebContents* GetAssociatedWebContents() const override;