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

Unified Diff: chrome/common/extensions/manifest_tests/extension_manifests_sandboxed_unittest.cc

Issue 15836003: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 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/manifest_tests/extension_manifests_sandboxed_unittest.cc
diff --git a/chrome/common/extensions/manifest_tests/extension_manifests_sandboxed_unittest.cc b/chrome/common/extensions/manifest_tests/extension_manifests_sandboxed_unittest.cc
index e6e2986c2cc9db71d3280a843c424a3170690e2a..da3965ce166fa78eb026e559462d8ce8ce9cb0a5 100644
--- a/chrome/common/extensions/manifest_tests/extension_manifests_sandboxed_unittest.cc
+++ b/chrome/common/extensions/manifest_tests/extension_manifests_sandboxed_unittest.cc
@@ -41,23 +41,30 @@ TEST_F(SandboxedPagesManifestTest, SandboxedPages) {
const char kCustomSandboxedCSP[] =
"sandbox; script-src: https://www.google.com";
- EXPECT_EQ(kSandboxedCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension1, "/test"));
- EXPECT_EQ(kDefaultCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension1, "/none"));
- EXPECT_EQ(kDefaultCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension2, "/test"));
- EXPECT_EQ(kCustomSandboxedCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension3, "/test"));
- EXPECT_EQ(kDefaultCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension3, "/none"));
- EXPECT_EQ(kSandboxedCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension4, "/test"));
EXPECT_EQ(
kSandboxedCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension5, "/path/test.ext"));
- EXPECT_EQ(kDefaultCSP,
- CSPInfo::GetResourceContentSecurityPolicy(extension5, "/test"));
+ CSPInfo::GetResourceContentSecurityPolicy(extension1.get(), "/test"));
+ EXPECT_EQ(
+ kDefaultCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension1.get(), "/none"));
+ EXPECT_EQ(
+ kDefaultCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension2.get(), "/test"));
+ EXPECT_EQ(
+ kCustomSandboxedCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension3.get(), "/test"));
+ EXPECT_EQ(
+ kDefaultCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension3.get(), "/none"));
+ EXPECT_EQ(
+ kSandboxedCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension4.get(), "/test"));
+ EXPECT_EQ(kSandboxedCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension5.get(),
+ "/path/test.ext"));
+ EXPECT_EQ(
+ kDefaultCSP,
+ CSPInfo::GetResourceContentSecurityPolicy(extension5.get(), "/test"));
Testcase testcases[] = {
Testcase("sandboxed_pages_invalid_1.json",

Powered by Google App Engine
This is Rietveld 408576698