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

Unified Diff: chrome/common/extensions/extension_unittest.cc

Issue 10213002: Make downloads.download() respect host permissions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: " Created 8 years, 7 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
Index: chrome/common/extensions/extension_unittest.cc
diff --git a/chrome/common/extensions/extension_unittest.cc b/chrome/common/extensions/extension_unittest.cc
index d0d6a7f91c6b27dbdbceb2b9a9694bb830bf9fbd..9f3060eeda0508d7b6f688b86839cb7918f3abfd 100644
--- a/chrome/common/extensions/extension_unittest.cc
+++ b/chrome/common/extensions/extension_unittest.cc
@@ -739,6 +739,13 @@ TEST_F(ExtensionScriptAndCaptureVisibleTest, Permissions) {
EXPECT_FALSE(extension->HasHostPermission(settings_url));
EXPECT_FALSE(extension->HasHostPermission(about_url));
EXPECT_TRUE(extension->HasHostPermission(favicon_url));
+ EXPECT_TRUE(extension->HasHostPermission(extension->url()));
+
+ GURL filesystem_url("filesystem:chrome-extension://" + extension->id() +
+ "/temp/dir/file.ext");
+ CHECK(filesystem_url.SchemeIsFileSystem());
+ CHECK(filesystem_url.inner_url());
+ EXPECT_TRUE(extension->HasHostPermission(filesystem_url));
// Test * for scheme, which implies just the http/https schemes.
extension = LoadManifestStrict("script_and_capture",

Powered by Google App Engine
This is Rietveld 408576698