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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/common/feature_policy/feature_policy_unittest.cc

Issue 2727803004: Replace string by enum in WebParsedFeaturePolicyDeclaration#feature (Closed)
Patch Set: Set upstream Created 3 years, 10 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
« content/common/feature_policy/feature_policy.cc ('K') | « content/common/feature_policy/feature_policy.cc ('k') | content/common/frame_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/feature_policy/feature_policy_unittest.cc
diff --git a/content/common/feature_policy/feature_policy_unittest.cc b/content/common/feature_policy/feature_policy_unittest.cc
index f19cc8883337b427ce34d98eb714b6fd982e42bb..faf54c92ada42cc4e801be51f4f77ab7201c5941 100644
--- a/content/common/feature_policy/feature_policy_unittest.cc
+++ b/content/common/feature_policy/feature_policy_unittest.cc
@@ -14,18 +14,18 @@ namespace {
// This is an example of a feature which should be enabled by default in all
// frames.
const FeaturePolicy::Feature kDefaultOnFeatureDfn{
- "default-on", FeaturePolicy::FeatureDefault::EnableForAll};
+ kDefaultOnFeature, FeaturePolicy::FeatureDefault::EnableForAll};
// This is an example of a feature which should be enabled in top-level frames,
// and same-origin child-frames, but must be delegated to all cross-origin
// frames explicitly.
const FeaturePolicy::Feature kDefaultSelfFeatureDfn{
- "default-self", FeaturePolicy::FeatureDefault::EnableForSelf};
+ kDefaultSelfFeature, FeaturePolicy::FeatureDefault::EnableForSelf};
// This is an example of a feature which should be disabled by default, both in
// top-level and nested frames.
const FeaturePolicy::Feature kDefaultOffFeatureDfn{
- "default-off", FeaturePolicy::FeatureDefault::DisableForAll};
+ kDefaultOffFeature, FeaturePolicy::FeatureDefault::DisableForAll};
// Define the three new features for testing
blink::WebFeaturePolicyFeature kDefaultOnFeature =
@@ -145,7 +145,7 @@ TEST_F(FeaturePolicyTest, TestCrossOriginChildCannotEnableFeature) {
CreateFromParentPolicy(nullptr, origin_a_);
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
- policy2->SetHeaderPolicy({{{"default-self", false, {origin_b_}}}});
+ policy2->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_b_}}}});
EXPECT_FALSE(policy2->IsFeatureEnabled(kDefaultSelfFeature));
}
@@ -167,7 +167,7 @@ TEST_F(FeaturePolicyTest, TestFrameSelfInheritance) {
// they are at a different origin.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_a_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_a_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -199,7 +199,7 @@ TEST_F(FeaturePolicyTest, TestReflexiveFrameSelfInheritance) {
// it is embedded by frame 2, for which the feature is not enabled.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_a_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -226,7 +226,7 @@ TEST_F(FeaturePolicyTest, TestSelectiveFrameInheritance) {
// enabled.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_b_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -247,7 +247,7 @@ TEST_F(FeaturePolicyTest, TestPolicyCanBlockSelf) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
policy1->SetHeaderPolicy(
- {{{"default-on", false, std::vector<url::Origin>()}}});
+ {{{kDefaultOnFeature, false, std::vector<url::Origin>()}}});
EXPECT_FALSE(policy1->IsFeatureEnabled(kDefaultOnFeature));
}
@@ -264,7 +264,7 @@ TEST_F(FeaturePolicyTest, TestParentPolicyBlocksSameOriginChildPolicy) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
policy1->SetHeaderPolicy(
- {{{"default-on", false, std::vector<url::Origin>()}}});
+ {{{kDefaultOnFeature, false, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_a_);
EXPECT_FALSE(policy2->IsFeatureEnabled(kDefaultOnFeature));
@@ -285,7 +285,7 @@ TEST_F(FeaturePolicyTest, TestChildPolicyCanBlockSelf) {
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
policy2->SetHeaderPolicy(
- {{{"default-on", false, std::vector<url::Origin>()}}});
+ {{{kDefaultOnFeature, false, std::vector<url::Origin>()}}});
EXPECT_FALSE(policy2->IsFeatureEnabled(kDefaultOnFeature));
}
@@ -308,7 +308,7 @@ TEST_F(FeaturePolicyTest, TestChildPolicyCanBlockChildren) {
CreateFromParentPolicy(nullptr, origin_a_);
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
- policy2->SetHeaderPolicy({{{"default-on", false, {origin_b_}}}});
+ policy2->SetHeaderPolicy({{{kDefaultOnFeature, false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_c_);
EXPECT_TRUE(policy2->IsFeatureEnabled(kDefaultOnFeature));
@@ -328,7 +328,7 @@ TEST_F(FeaturePolicyTest, TestParentPolicyBlocksCrossOriginChildPolicy) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
policy1->SetHeaderPolicy(
- {{{"default-on", false, std::vector<url::Origin>()}}});
+ {{{kDefaultOnFeature, false, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
EXPECT_FALSE(policy2->IsFeatureEnabled(kDefaultOnFeature));
@@ -351,7 +351,7 @@ TEST_F(FeaturePolicyTest, TestEnableForAllOrigins) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
policy1->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -377,7 +377,7 @@ TEST_F(FeaturePolicyTest, TestDefaultOnEnablesForAllAncestors) {
// Feature should be disabled in frame 1; enabled in frames 2, 3 and 4.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-on", false, {origin_b_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultOnFeature, false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -406,7 +406,7 @@ TEST_F(FeaturePolicyTest, TestDefaultSelfRespectsSameOriginEmbedding) {
// Feature should be disabled in frames 1 and 4; enabled in frames 2 and 3.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_b_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -435,15 +435,15 @@ TEST_F(FeaturePolicyTest, TestDefaultOffMustBeDelegatedToAllCrossOriginFrames) {
// Feature should be disabled in frames 1, 3 and 4; enabled in frame 2 only.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-off", false, {origin_b_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
- policy2->SetHeaderPolicy({{{"default-off", false, {origin_b_}}}});
+ policy2->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy4 =
CreateFromParentPolicy(policy2.get(), origin_c_);
- policy4->SetHeaderPolicy({{{"default-off", false, {origin_c_}}}});
+ policy4->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_c_}}}});
EXPECT_FALSE(policy1->IsFeatureEnabled(kDefaultOffFeature));
EXPECT_TRUE(policy2->IsFeatureEnabled(kDefaultOffFeature));
EXPECT_FALSE(policy3->IsFeatureEnabled(kDefaultOffFeature));
@@ -467,11 +467,11 @@ TEST_F(FeaturePolicyTest, TestReenableForAllOrigins) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
policy1->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
policy2->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_a_);
EXPECT_TRUE(policy1->IsFeatureEnabled(kDefaultSelfFeature));
@@ -495,11 +495,11 @@ TEST_F(FeaturePolicyTest, TestBlockedFrameCannotReenable) {
// Feature should be enabled at the top level; disabled in all other frames.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_a_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
policy2->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_a_);
std::unique_ptr<FeaturePolicy> policy4 =
@@ -526,10 +526,12 @@ TEST_F(FeaturePolicyTest, TestEnabledFrameCanDelegate) {
// Feature should be enabled in all frames.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_, origin_b_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultSelfFeature false, {origin_a_, origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
- policy2->SetHeaderPolicy({{{"default-self", false, {origin_b_, origin_c_}}}});
+ policy2->SetHeaderPolicy(
+ {{{kDefaultSelfFeature false, {origin_b_, origin_c_}}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_c_);
EXPECT_TRUE(policy1->IsFeatureEnabled(kDefaultSelfFeature));
@@ -553,7 +555,8 @@ TEST_F(FeaturePolicyTest, TestEnabledFrameCanDelegateByDefault) {
// Feature should be enabled in frames 1, 2, and 3, and disabled in frame 4.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-on", false, {origin_a_, origin_b_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultOnFeature, false, {origin_a_, origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -583,7 +586,8 @@ TEST_F(FeaturePolicyTest, TestNonNestedFeaturesDontDelegateByDefault) {
// 4.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_, origin_b_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultSelfFeature false, {origin_a_, origin_b_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
std::unique_ptr<FeaturePolicy> policy3 =
@@ -615,13 +619,14 @@ TEST_F(FeaturePolicyTest, TestFeaturesAreIndependent) {
// should be enabled in frame 1, and disabled in frames 2 and 3.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_, origin_b_}},
- {"default-on", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultSelfFeature false, {origin_a_, origin_b_}},
+ {kDefaultOnFeature, false, {origin_a_}}}});
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentPolicy(policy1.get(), origin_b_);
policy2->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()},
- {"default-on", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()},
+ {kDefaultOnFeature, true, std::vector<url::Origin>()}}});
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentPolicy(policy2.get(), origin_c_);
EXPECT_TRUE(policy1->IsFeatureEnabled(kDefaultSelfFeature));
@@ -641,7 +646,8 @@ TEST_F(FeaturePolicyTest, TestFeatureEnabledForOrigin) {
// and disabled for origin C.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-off", false, {origin_a_, origin_b_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultOffFeature, false, {origin_a_, origin_b_}}}});
EXPECT_TRUE(
policy1->IsFeatureEnabledForOrigin(kDefaultOffFeature, origin_a_));
EXPECT_TRUE(
@@ -670,7 +676,7 @@ TEST_F(FeaturePolicyTest, TestSimpleFramePolicy) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
ParsedFeaturePolicyHeader frame_policy = {
- {{"default-self", false, {origin_b_}}}};
+ {{kDefaultSelfFeature false, {origin_b_}}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy, origin_b_);
EXPECT_TRUE(
@@ -705,7 +711,7 @@ TEST_F(FeaturePolicyTest, TestAllOriginFramePolicy) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
ParsedFeaturePolicyHeader frame_policy = {
- {{"default-self", true, std::vector<url::Origin>()}}};
+ {{kDefaultSelfFeature true, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy, origin_b_);
EXPECT_TRUE(
@@ -751,11 +757,11 @@ TEST_F(FeaturePolicyTest, TestFramePolicyCanBeFurtherDelegated) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-self", false, {origin_b_}}}};
+ {{kDefaultSelfFeature false, {origin_b_}}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_b_);
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-self", false, {origin_c_}}}};
+ {{kDefaultSelfFeature false, {origin_c_}}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy2.get(), frame_policy2, origin_c_);
std::unique_ptr<FeaturePolicy> policy4 =
@@ -796,11 +802,11 @@ TEST_F(FeaturePolicyTest, TestDefaultOnCanBeDisabledByFramePolicy) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-on", false, std::vector<url::Origin>()}}};
+ {{kDefaultOnFeature, false, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_a_);
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-on", false, std::vector<url::Origin>()}}};
+ {{kDefaultOnFeature, false, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy2, origin_b_);
EXPECT_TRUE(policy1->IsFeatureEnabledForOrigin(kDefaultOnFeature, origin_a_));
@@ -841,13 +847,13 @@ TEST_F(FeaturePolicyTest, TestDefaultOffMustBeEnabledByChildFrame) {
// child frames because they did not declare their own policy to enable it.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-off", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_a_}}}});
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-off", false, {origin_a_}}}};
+ {{kDefaultOffFeature, false, {origin_a_}}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_a_);
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-off", false, {origin_b_}}}};
+ {{kDefaultOffFeature, false, {origin_b_}}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy2, origin_b_);
EXPECT_TRUE(
@@ -892,17 +898,17 @@ TEST_F(FeaturePolicyTest, TestDefaultOffCanBeEnabledByChildFrame) {
// they declare their own policy to enable it.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-off", false, {origin_a_}}}});
+ policy1->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_a_}}}});
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-off", false, {origin_a_}}}};
+ {{kDefaultOffFeature, false, {origin_a_}}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_a_);
- policy2->SetHeaderPolicy({{{"default-off", false, {origin_a_}}}});
+ policy2->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_a_}}}});
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-off", false, {origin_b_}}}};
+ {{kDefaultOffFeature, false, {origin_b_}}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy2, origin_b_);
- policy3->SetHeaderPolicy({{{"default-off", false, {origin_b_}}}});
+ policy3->SetHeaderPolicy({{{kDefaultOffFeature, false, {origin_b_}}}});
EXPECT_TRUE(
policy1->IsFeatureEnabledForOrigin(kDefaultOffFeature, origin_a_));
EXPECT_FALSE(
@@ -946,16 +952,17 @@ TEST_F(FeaturePolicyTest, TestFramePolicyModifiesHeaderPolicy) {
// policy.
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
- policy1->SetHeaderPolicy({{{"default-self", false, {origin_a_, origin_b_}}}});
+ policy1->SetHeaderPolicy(
+ {{{kDefaultSelfFeature false, {origin_a_, origin_b_}}}});
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-self", false, std::vector<url::Origin>()}}};
+ {{kDefaultSelfFeature false, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_b_);
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-self", false, std::vector<url::Origin>()}}};
+ {{kDefaultSelfFeature false, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy2, origin_b_);
- policy3->SetHeaderPolicy({{{"default-self", false, {origin_b_}}}});
+ policy3->SetHeaderPolicy({{{kDefaultSelfFeature false, {origin_b_}}}});
EXPECT_FALSE(
policy2->IsFeatureEnabledForOrigin(kDefaultSelfFeature, origin_b_));
EXPECT_FALSE(
@@ -990,13 +997,13 @@ TEST_F(FeaturePolicyTest, TestCombineFrameAndHeaderPolicies) {
std::unique_ptr<FeaturePolicy> policy1 =
CreateFromParentPolicy(nullptr, origin_a_);
ParsedFeaturePolicyHeader frame_policy1 = {
- {{"default-self", false, {origin_b_}}}};
+ {{kDefaultSelfFeature false, {origin_b_}}}};
std::unique_ptr<FeaturePolicy> policy2 =
CreateFromParentWithFramePolicy(policy1.get(), frame_policy1, origin_b_);
policy2->SetHeaderPolicy(
- {{{"default-self", true, std::vector<url::Origin>()}}});
+ {{{kDefaultSelfFeature true, std::vector<url::Origin>()}}});
ParsedFeaturePolicyHeader frame_policy2 = {
- {{"default-self", false, std::vector<url::Origin>()}}};
+ {{kDefaultSelfFeature false, std::vector<url::Origin>()}}};
std::unique_ptr<FeaturePolicy> policy3 =
CreateFromParentWithFramePolicy(policy2.get(), frame_policy2, origin_c_);
std::unique_ptr<FeaturePolicy> policy4 =
« content/common/feature_policy/feature_policy.cc ('K') | « content/common/feature_policy/feature_policy.cc ('k') | content/common/frame_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698