Fix message routing for BrowserPlugin in iframe

chrome/browser/apps/web_view_browsertest.cc

Index: chrome/browser/apps/web_view_browsertest.cc
diff --git a/chrome/browser/apps/web_view_browsertest.cc b/chrome/browser/apps/web_view_browsertest.cc
index eaaa40afd78abb53dd1d516ad7f699fb0e01e4cd..7f2b1834c64e830a5ec275beea48f93508532b1a 100644
--- a/chrome/browser/apps/web_view_browsertest.cc
+++ b/chrome/browser/apps/web_view_browsertest.cc
@@ -2424,6 +2424,13 @@ IN_PROC_BROWSER_TEST_F(WebViewTest, Shim_WebViewWebRequestRegistryHasNoCache) {
   EXPECT_FALSE(registry->rules_cache_delegate_for_testing());
 }
 
+// This test verify that the set of rules registries of a webview will be

[Charlie Reis, 2014/12/18 00:27:40]

nit: verifies

[Fady Samuel, 2014/12/18 00:35:50]

Done.

+// removed from RulesRegistryService after the webview is gone.
+// http://crbug.com/438327

[Charlie Reis, 2014/12/18 00:27:40]

This is a different bug number.  Is it a related bug, or just a typo?

Also the comment should probably mention that this is for the iframe case, since
it's the main emphasis.

[Fady Samuel, 2014/12/18 00:35:50]

Doh. Blind copy and paste. Fixed.

+IN_PROC_BROWSER_TEST_F(WebViewTest, Shim_TestWebViewInsideFrame) {
+  LoadAppWithGuest("web_view/inside_iframe");

[Charlie Reis, 2014/12/18 00:27:39]

Does this file need to be added?

[Fady Samuel, 2014/12/18 00:35:50]

Yes.

+}
+
 // <webview> screenshot capture fails with ubercomp.
 // See http://crbug.com/327035.
 IN_PROC_BROWSER_TEST_F(WebViewCaptureTest,