Skip to content

Commit

Permalink
Disable indexing features to allow BravePrefProvider work as intended.
Browse files Browse the repository at this point in the history
Upstream change:
https://chromium.googlesource.com/chromium/src/+/cdb4efcd0b060341a2864a63d6104d91082e8064

commit cdb4efcd0b060341a2864a63d6104d91082e8064
Author: Patricia Alfonso <trishalfonso@google.com>
Date:   Thu Apr 11 22:38:05 2024 +0000

    [optimize-content-settings] Enable indexed content settings by default

    Bug: 316530672
    Change-Id: I33b4852e9d2ba1bcade6b1e6c809a6c6a9d67236
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5435975
    Reviewed-by: Christian Dullweber <dullweber@chromium.org>
    Reviewed-by: Filipa Senra <fsenra@google.com>
    Commit-Queue: Patricia Alfonso <trishalfonso@google.com>
    Cr-Commit-Position: refs/heads/main@{#1286152}
  • Loading branch information
goodov committed Apr 26, 2024
1 parent 6bcc331 commit bf720c3
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ namespace features {
OVERRIDE_FEATURE_DEFAULT_STATES({{
{kTrackingProtection3pcd, base::FEATURE_DISABLED_BY_DEFAULT},
{kUserBypassUI, base::FEATURE_DISABLED_BY_DEFAULT},

// Disable content settings indexing as it's currently doesn't work well
// with our BravePrefProvider.
// TODO(https://github.com/brave/brave-browser/issues/37877).
{kHostIndexedMetadataGrants, base::FEATURE_DISABLED_BY_DEFAULT},
{kIndexedHostContentSettingsMap, base::FEATURE_DISABLED_BY_DEFAULT},
}});

} // namespace features
Expand Down

0 comments on commit bf720c3

Please sign in to comment.