Change wildcard source expression matching to conform latest spec

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 dca4eb548b3dc23d9bee072b00e834213c2fa205..7f2262de713cdeba77907858b9eafdefe38fcaf5 100644
--- a/chrome/browser/pdf/pdf_extension_test.cc
+++ b/chrome/browser/pdf/pdf_extension_test.cc
@@ -464,6 +464,18 @@ IN_PROC_BROWSER_TEST_F(PDFExtensionTest, EnsureSameOriginRepliesAllowed) {
                            true);
 }
 
+// This test ensures that PDF can be loaded from local file
+IN_PROC_BROWSER_TEST_F(PDFExtensionTest, EnsurePDFFromLocalFileLoads) {
+  base::FilePath test_data_dir;
+  ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_dir));
+  test_data_dir = test_data_dir.AppendASCII("pdf");
+  test_data_dir = test_data_dir.AppendASCII("test.pdf");

[Lei Zhang, 2016/08/09 00:28:45]

Maybe make a new base::FilePath test_data_file variable here, so the 2 lines
below can refer to test_data_file and it reads a little bit better.

[Sergey Shekyan, 2016/08/09 00:38:28]

Done.

[Sergey Shekyan, 2016/08/09 00:38:28]

Acknowledged.

+  ASSERT_TRUE(PathExists(test_data_dir));
+  GURL test_pdf_url("file://" + test_data_dir.MaybeAsASCII());
+  content::WebContents* guest_contents = LoadPdfGetGuestContents(test_pdf_url);
+  ASSERT_TRUE(guest_contents);
+}
+
 // This test ensures that link permissions are enforced properly in PDFs.
 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, LinkPermissions) {
   GURL test_pdf_url(embedded_test_server()->GetURL("/pdf/test.pdf"));