Commit ceb30f4b authored by Sean Harrison's avatar Sean Harrison Committed by Chromium LUCI CQ

[Privacy Sandbox] Fix "policy" setting

Add is_int check before getting the default cookie policy to only grab
the value when it exists and fix a badVariantAccess.

Bug: 1167622
Change-Id: I0733a1c61233aa702cc40698b9d06c2878222ec6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631501Reviewed-by: default avatarMartin Šrámek <msramek@chromium.org>
Commit-Queue: Sean Harrison <harrisonsean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844499}
parent 4f89b991
...@@ -41,7 +41,7 @@ void CookieSettingsPolicyHandler::ApplyPolicySettings( ...@@ -41,7 +41,7 @@ void CookieSettingsPolicyHandler::ApplyPolicySettings(
// privacy sandbox if it is set to BLOCK. // privacy sandbox if it is set to BLOCK.
const base::Value* default_cookie_setting = const base::Value* default_cookie_setting =
policies.GetValue(policy::key::kDefaultCookiesSetting); policies.GetValue(policy::key::kDefaultCookiesSetting);
if (default_cookie_setting && if (default_cookie_setting && default_cookie_setting->is_int() &&
static_cast<ContentSetting>(default_cookie_setting->GetInt()) == static_cast<ContentSetting>(default_cookie_setting->GetInt()) ==
CONTENT_SETTING_BLOCK) { CONTENT_SETTING_BLOCK) {
prefs->SetBoolean(prefs::kPrivacySandboxApisEnabled, false); prefs->SetBoolean(prefs::kPrivacySandboxApisEnabled, false);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment