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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/common/extensions/extension.h" 5 #include "chrome/common/extensions/extension.h"
6 6
7 #include "base/format_macros.h" 7 #include "base/format_macros.h"
8 #include "base/file_path.h" 8 #include "base/file_path.h"
9 #include "base/file_util.h" 9 #include "base/file_util.h"
10 #include "base/json/json_file_value_serializer.h" 10 #include "base/json/json_file_value_serializer.h"
(...skipping 721 matching lines...) Expand 10 before | Expand all | Expand 10 after
732 EXPECT_TRUE(Allowed(extension, https_url)); 732 EXPECT_TRUE(Allowed(extension, https_url));
733 EXPECT_TRUE(Blocked(extension, file_url)); 733 EXPECT_TRUE(Blocked(extension, file_url));
734 EXPECT_TRUE(Blocked(extension, settings_url)); 734 EXPECT_TRUE(Blocked(extension, settings_url));
735 EXPECT_TRUE(CaptureOnly(extension, favicon_url)); 735 EXPECT_TRUE(CaptureOnly(extension, favicon_url));
736 EXPECT_TRUE(Blocked(extension, about_url)); 736 EXPECT_TRUE(Blocked(extension, about_url));
737 EXPECT_TRUE(Blocked(extension, extension_url)); 737 EXPECT_TRUE(Blocked(extension, extension_url));
738 738
739 EXPECT_FALSE(extension->HasHostPermission(settings_url)); 739 EXPECT_FALSE(extension->HasHostPermission(settings_url));
740 EXPECT_FALSE(extension->HasHostPermission(about_url)); 740 EXPECT_FALSE(extension->HasHostPermission(about_url));
741 EXPECT_TRUE(extension->HasHostPermission(favicon_url)); 741 EXPECT_TRUE(extension->HasHostPermission(favicon_url));
742 EXPECT_TRUE(extension->HasHostPermission(extension->url()));
743
744 GURL filesystem_url("filesystem:chrome-extension://" + extension->id() +
745 "/temp/dir/file.ext");
746 CHECK(filesystem_url.SchemeIsFileSystem());
747 CHECK(filesystem_url.inner_url());
748 EXPECT_TRUE(extension->HasHostPermission(filesystem_url));
742 749
743 // Test * for scheme, which implies just the http/https schemes. 750 // Test * for scheme, which implies just the http/https schemes.
744 extension = LoadManifestStrict("script_and_capture", 751 extension = LoadManifestStrict("script_and_capture",
745 "extension_wildcard.json"); 752 "extension_wildcard.json");
746 EXPECT_TRUE(Allowed(extension, http_url)); 753 EXPECT_TRUE(Allowed(extension, http_url));
747 EXPECT_TRUE(Allowed(extension, https_url)); 754 EXPECT_TRUE(Allowed(extension, https_url));
748 EXPECT_TRUE(Blocked(extension, settings_url)); 755 EXPECT_TRUE(Blocked(extension, settings_url));
749 EXPECT_TRUE(Blocked(extension, about_url)); 756 EXPECT_TRUE(Blocked(extension, about_url));
750 EXPECT_TRUE(Blocked(extension, file_url)); 757 EXPECT_TRUE(Blocked(extension, file_url));
751 EXPECT_TRUE(Blocked(extension, favicon_url)); 758 EXPECT_TRUE(Blocked(extension, favicon_url));
(...skipping 207 matching lines...) Expand 10 before | Expand all | Expand 10 after
959 } 966 }
960 967
961 TEST(ExtensionTest, GetSyncTypeExtensionWithTwoPlugins) { 968 TEST(ExtensionTest, GetSyncTypeExtensionWithTwoPlugins) {
962 scoped_refptr<Extension> extension( 969 scoped_refptr<Extension> extension(
963 MakeSyncTestExtension(EXTENSION, GURL(), GURL(), 970 MakeSyncTestExtension(EXTENSION, GURL(), GURL(),
964 Extension::INTERNAL, 2, FilePath())); 971 Extension::INTERNAL, 2, FilePath()));
965 if (extension) 972 if (extension)
966 EXPECT_EQ(extension->GetSyncType(), Extension::SYNC_TYPE_NONE); 973 EXPECT_EQ(extension->GetSyncType(), Extension::SYNC_TYPE_NONE);
967 } 974 }
968 #endif // !defined(OS_CHROMEOS) 975 #endif // !defined(OS_CHROMEOS)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698