Add test to ensure that URLs that redirect inside the PDF plugin fail to load

chrome/browser/pdf/pdf_extension_test.cc

Index: chrome/browser/pdf/pdf_extension_test.cc
diff --git a/chrome/browser/pdf/pdf_extension_test.cc b/chrome/browser/pdf/pdf_extension_test.cc
index 118da0eb14645ffc82b6a45bc2101669ca25a353..f32708e417046df7e1d4bbee0fdd0cbf00d75791 100644
--- a/chrome/browser/pdf/pdf_extension_test.cc
+++ b/chrome/browser/pdf/pdf_extension_test.cc
@@ -910,3 +910,10 @@ IN_PROC_BROWSER_TEST_F(PDFExtensionTest, LinkShiftLeftClick) {
   const GURL& url = active_web_contents->GetURL();
   ASSERT_EQ(std::string("http://www.example.com/"), url.spec());
 }
+
+// Test that if the plugin tries to load a URL that redirects that it will fail

[Lei Zhang, 2016/11/08 01:28:10]

... _then_ it will fail?

[raymes, 2016/11/08 06:00:01]

Done.

+// to load. This is to avoid the source origin of the document changing during
+// the redirect, which can have security implications. crbug.com/653749

[Lei Zhang, 2016/11/08 01:28:10]

If you add https:// in the URL, then cs.chromium.org will auto-linkify, which is
nice.

[raymes, 2016/11/08 06:00:01]

Done.

+IN_PROC_BROWSER_TEST_F(PDFExtensionTest, RedirectsFailInPlugin) {
+  RunTestsInFile("redirects_fail_test.js", "test.pdf");
+}