Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1021)

Unified Diff: chrome/browser/pdf/pdf_extension_test.cc

Issue 2407713002: Limit PDF helper extension to print preview only (Closed)
Patch Set: Use location.origin instead of location.ancestorOrigins. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/browser_api.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..74f8a17e9ee227943c303474307bc91e4148e893 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 be loaded directly.
+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,
+ "*Streams are only available from a mime handler view guest.*"));
+ 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;
« no previous file with comments | « no previous file | chrome/browser/resources/pdf/browser_api.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698