Fix MimeHandlerView plugin loading for data: URLs

extensions/browser/guest_view/mime_handler_view/mime_handler_view_browsertest.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 "base/base_paths.h"
 #include "base/files/file_util.h"
 #include "base/path_service.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/test/browser_test_utils.h"
@@ skipping 12 matching lines @@
     const extensions::Extension* extension =
         LoadExtension(test_data_dir_.AppendASCII("mime_handler_view"));
     if (!extension)
       return nullptr;
 
     CHECK_EQ(std::string(kExtensionId), extension->id());
 
     return extension;
   }
 
-  void RunTest(const std::string& path) {
+  void RunTestWithUrl(const GURL& url) {
     const extensions::Extension* extension = LoadTestExtension();
     ASSERT_TRUE(extension);
 
+    extensions::ResultCatcher catcher;
+    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"));
 
-    extensions::ResultCatcher catcher;
-
-    ui_test_utils::NavigateToURL(browser(),
-                                 embedded_test_server()->GetURL("/" + path));
-
-    if (!catcher.GetNextResult())
-      FAIL() << catcher.message();
+    RunTestWithUrl(embedded_test_server()->GetURL("/" + path));
   }
 };
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, PostMessage) {
   RunTest("test_postmessage.html");
 }
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, Basic) {
   RunTest("testBasic.csv");
 }
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, Embedded) {
   RunTest("test_embedded.html");
 }
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, Iframe) {
   RunTest("test_iframe.html");
 }
 
 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");
+}