Limit PDF helper extension to print preview only

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 d2642f638922cc4e51d996396728f29be62ae6eb..e1bf45df61f39ca45c14e0382005ec8f8e18598d 100644
--- a/chrome/browser/pdf/pdf_extension_test.cc
+++ b/chrome/browser/pdf/pdf_extension_test.cc
@@ -464,6 +464,24 @@ IN_PROC_BROWSER_TEST_F(PDFExtensionTest, EnsureSameOriginRepliesAllowed) {
                            true);
 }
 
+// Ensure that the PDF component extension cannot directly be loaded.

[nasko, 2016/10/11 21:26:33]

nit: s/cannot directly be loaded/cannot be loaded directly/.

[robwu, 2016/10/11 22:15:46]

Done.

+IN_PROC_BROWSER_TEST_F(PDFExtensionTest, BlockDirectAccess) {
+  content::WebContents* web_contents =
+    browser()->tab_strip_model()->GetActiveWebContents();
+
+  std::unique_ptr<content::ConsoleObserverDelegate> console_delegate(
+      new content::ConsoleObserverDelegate(
+        web_contents,

[nasko, 2016/10/11 21:26:33]

nit: Shouldn't this be 4 spaces indent? git cl format should help ensure proper
formatting.

[robwu, 2016/10/11 22:15:46]

Yes, four it is. Done!

+        "*Refused to load the standalone extension.*"));
+  web_contents->SetDelegate(console_delegate.get());
+  GURL forbiddenUrl(
+      "chrome-extension://mhjfbmdgcfjbbpaeojofohoefgiehjai/index.html?"
+      "https://example.com/notrequested.pdf");
+  ui_test_utils::NavigateToURL(browser(), forbiddenUrl);
+
+  console_delegate->Wait();
+}
+
 // This test ensures that PDF can be loaded from local file
 IN_PROC_BROWSER_TEST_F(PDFExtensionTest, EnsurePDFFromLocalFileLoads) {
   base::FilePath test_data_dir;