Fix MimeHandlerView plugin loading for data: URLs

extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc

Index: extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc
diff --git a/extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc
index 5b09caf3beeb4e9ff1b914dcf032c1b4b662fedb..7db592c10922b7346e25b12edb8fdc9ff6d6f53c 100644
--- a/extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc
+++ b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.cc
@@ -30,22 +30,24 @@ class MimeHandlerViewTest : public ExtensionApiTest {
     return extension;
   }
 
-  void RunTest(const std::string& path) {
+  void RunTestWithUrl(const GURL& url) {
     const extensions::Extension* extension = LoadTestExtension();
     ASSERT_TRUE(extension);
 
-    ASSERT_TRUE(StartEmbeddedTestServer());
-    embedded_test_server()->ServeFilesFromDirectory(
-        test_data_dir_.AppendASCII("mime_handler_view"));
-
     extensions::ResultCatcher catcher;
-
-    ui_test_utils::NavigateToURL(browser(),
-                                 embedded_test_server()->GetURL("/" + path));
+    ui_test_utils::NavigateToURL(browser(), url);
 
     if (!catcher.GetNextResult())
       FAIL() << catcher.message();
   }
+
+  void RunTest(const std::string& path) {
+    ASSERT_TRUE(StartEmbeddedTestServer());
+    embedded_test_server()->ServeFilesFromDirectory(
+        test_data_dir_.AppendASCII("mime_handler_view"));
+
+    RunTestWithUrl(embedded_test_server()->GetURL("/" + path));
+  }
 };
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, PostMessage) {
@@ -67,3 +69,12 @@ IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, Iframe) {
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, Abort) {
   RunTest("testAbort.csv");
 }
+
+IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, DataUrl) {
+  const char* kDataUrlCsv = "data:text/csv;base64,Y29udGVudCB0byByZWFkCg==";

[Sam McNally, 2015/04/22 04:05:03]

Would this navigation have bypassed the browser previously?

[raymes, 2015/04/22 04:11:48]

No, this one would have worked. The one below would have failed (I tested that
it did :).

+  RunTestWithUrl(GURL(kDataUrlCsv));
+}
+
+IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, EmbeddedDataUrl) {
+  RunTest("test_embedded_data_url.html");
+}