From: csagan5 <32685696+csagan5@users.noreply.github.com> Date: Sat, 2 May 2020 00:25:25 +0200 Subject: Modify default preferences License: GPL-3.0-only - https://spdx.org/licenses/GPL-3.0-only.html --- chrome/browser/about_flags.cc | 1 + .../extensions/background_mode_manager.cc | 2 +- .../browser/chrome_content_browser_client.cc | 2 +- chrome/browser/prefs/browser_prefs.cc | 2 +- chrome/browser/preloading/preloading_prefs.h | 2 +- chrome/browser/profiles/profile.cc | 2 +- .../reset_page/reset_profile_dialog.html | 2 +- chrome/browser/ui/browser_ui_prefs.cc | 4 ++-- components/bookmarks/browser/BUILD.gn | 2 ++ components/bookmarks/browser/bookmark_utils.cc | 7 +++++-- components/bookmarks/browser/features.cc | 17 +++++++++++++++++ components/bookmarks/browser/features.h | 18 ++++++++++++++++++ .../core/browser/content_settings_registry.cc | 8 ++++---- .../core/browser/cookie_settings.cc | 2 +- components/payments/core/payment_prefs.cc | 2 +- .../core/common/safe_browsing_prefs.cc | 4 ++-- .../security_interstitials/core/features.cc | 1 + .../primary_account_manager.cc | 2 +- .../Modify-default-preferences.inc | 8 ++++++++ .../features_cc/Modify-default-preferences.inc | 1 + 20 files changed, 70 insertions(+), 19 deletions(-) create mode 100644 components/bookmarks/browser/features.cc create mode 100644 components/bookmarks/browser/features.h create mode 100755 cromite_flags/chrome/browser/about_flags_cc/Modify-default-preferences.inc create mode 100755 cromite_flags/third_party/blink/common/features_cc/Modify-default-preferences.inc diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc --- a/chrome/browser/about_flags.cc +++ b/chrome/browser/about_flags.cc @@ -89,6 +89,7 @@ #include "components/autofill/core/common/autofill_payments_features.h" #include "components/autofill/core/common/autofill_switches.h" #include "components/autofill/core/common/autofill_util.h" +#include "components/bookmarks/browser/features.h" #include "components/browser_sync/browser_sync_switches.h" #include "components/browser_ui/accessibility/android/features.h" #include "components/browsing_data/core/features.h" diff --git a/chrome/browser/background/extensions/background_mode_manager.cc b/chrome/browser/background/extensions/background_mode_manager.cc --- a/chrome/browser/background/extensions/background_mode_manager.cc +++ b/chrome/browser/background/extensions/background_mode_manager.cc @@ -348,7 +348,7 @@ BackgroundModeManager::~BackgroundModeManager() { // static void BackgroundModeManager::RegisterPrefs(PrefRegistrySimple* registry) { - registry->RegisterBooleanPref(prefs::kBackgroundModeEnabled, true); + registry->RegisterBooleanPref(prefs::kBackgroundModeEnabled, false); } void BackgroundModeManager::RegisterProfile(Profile* profile) { diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc --- a/chrome/browser/chrome_content_browser_client.cc +++ b/chrome/browser/chrome_content_browser_client.cc @@ -1541,7 +1541,7 @@ void ChromeContentBrowserClient::RegisterLocalStatePrefs( void ChromeContentBrowserClient::RegisterProfilePrefs( user_prefs::PrefRegistrySyncable* registry) { registry->RegisterBooleanPref(prefs::kDisable3DAPIs, false); - registry->RegisterBooleanPref(prefs::kEnableHyperlinkAuditing, true); + registry->RegisterBooleanPref(prefs::kEnableHyperlinkAuditing, false); // Register user prefs for mapping SitePerProcess and IsolateOrigins in // user policy in addition to the same named ones in Local State (which are // used for mapping the command-line flags). diff --git a/chrome/browser/prefs/browser_prefs.cc b/chrome/browser/prefs/browser_prefs.cc --- a/chrome/browser/prefs/browser_prefs.cc +++ b/chrome/browser/prefs/browser_prefs.cc @@ -1271,7 +1271,7 @@ void RegisterProfilePrefsForMigration( registry->RegisterTimePref(kMerchantTrustPageInfoLastOpenTime, base::Time()); // Deprecated 12/2025. - registry->RegisterBooleanPref(kCloudPrintProxyEnabled, true); + registry->RegisterBooleanPref(kCloudPrintProxyEnabled, false); registry->RegisterStringPref(kCloudPrintEmail, std::string()); #if BUILDFLAG(IS_ANDROID) diff --git a/chrome/browser/preloading/preloading_prefs.h b/chrome/browser/preloading/preloading_prefs.h --- a/chrome/browser/preloading/preloading_prefs.h +++ b/chrome/browser/preloading/preloading_prefs.h @@ -23,7 +23,7 @@ enum class NetworkPredictionOptions { kWifiOnlyDeprecated = 1, kDisabled = 2, kExtended = 3, - kDefault = kWifiOnlyDeprecated, + kDefault = kDisabled, }; // Enum representing possible values of the Preload Pages opt-in state. These diff --git a/chrome/browser/profiles/profile.cc b/chrome/browser/profiles/profile.cc --- a/chrome/browser/profiles/profile.cc +++ b/chrome/browser/profiles/profile.cc @@ -328,7 +328,7 @@ const char Profile::kProfileKey[] = "__PROFILE__"; void Profile::RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) { registry->RegisterBooleanPref( prefs::kSearchSuggestEnabled, - true, + false, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); #if BUILDFLAG(IS_ANDROID) registry->RegisterStringPref( diff --git a/chrome/browser/resources/settings/reset_page/reset_profile_dialog.html b/chrome/browser/resources/settings/reset_page/reset_profile_dialog.html --- a/chrome/browser/resources/settings/reset_page/reset_profile_dialog.html +++ b/chrome/browser/resources/settings/reset_page/reset_profile_dialog.html @@ -33,7 +33,7 @@
- + $i18nRaw{resetPageFeedback}
diff --git a/chrome/browser/ui/browser_ui_prefs.cc b/chrome/browser/ui/browser_ui_prefs.cc --- a/chrome/browser/ui/browser_ui_prefs.cc +++ b/chrome/browser/ui/browser_ui_prefs.cc @@ -134,7 +134,7 @@ void RegisterBrowserUserPrefs(user_prefs::PrefRegistrySyncable* registry) { registry->RegisterBooleanPref(prefs::kWebAppCreateInAppsMenu, true); registry->RegisterBooleanPref(prefs::kWebAppCreateInQuickLaunchBar, true); registry->RegisterBooleanPref( - translate::prefs::kOfferTranslateEnabled, true, + translate::prefs::kOfferTranslateEnabled, false, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); registry->RegisterDictionaryPref(prefs::kBrowserWindowPlacement); registry->RegisterDictionaryPref(prefs::kBrowserWindowPlacementPopup); @@ -215,7 +215,7 @@ void RegisterBrowserUserPrefs(user_prefs::PrefRegistrySyncable* registry) { #endif registry->RegisterBooleanPref( - prefs::kHttpsOnlyModeEnabled, false, + prefs::kHttpsOnlyModeEnabled, true, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); registry->RegisterBooleanPref( prefs::kHttpsFirstBalancedMode, false, diff --git a/components/bookmarks/browser/BUILD.gn b/components/bookmarks/browser/BUILD.gn --- a/components/bookmarks/browser/BUILD.gn +++ b/components/bookmarks/browser/BUILD.gn @@ -24,6 +24,7 @@ static_library("browser") { "bookmark_undo_provider.h", "bookmark_utils.h", "bookmark_uuids.h", + "features.h", "history_bookmark_model.h", "model_loader.h", "scoped_group_bookmark_actions.h", @@ -47,6 +48,7 @@ static_library("browser") { "bookmark_storage.cc", "bookmark_utils.cc", "bookmark_uuids.cc", + "features.cc", "model_loader.cc", "scoped_group_bookmark_actions.cc", "titled_url_index.cc", diff --git a/components/bookmarks/browser/bookmark_utils.cc b/components/bookmarks/browser/bookmark_utils.cc --- a/components/bookmarks/browser/bookmark_utils.cc +++ b/components/bookmarks/browser/bookmark_utils.cc @@ -27,6 +27,7 @@ #include "components/bookmarks/browser/bookmark_client.h" #include "components/bookmarks/browser/bookmark_model.h" #include "components/bookmarks/browser/bookmark_node.h" +#include "components/bookmarks/browser/features.h" #include "components/bookmarks/browser/scoped_group_bookmark_actions.h" #include "components/bookmarks/common/bookmark_pref_names.h" #include "components/pref_registry/pref_registry_syncable.h" @@ -450,11 +451,13 @@ bool DoesBookmarkContainWords(const std::u16string& title, void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) { registry->RegisterBooleanPref( - prefs::kShowBookmarkBar, false, + prefs::kShowBookmarkBar, true, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); registry->RegisterBooleanPref(prefs::kEditBookmarksEnabled, true); registry->RegisterBooleanPref( - prefs::kShowAppsShortcutInBookmarkBar, false, + prefs::kShowAppsShortcutInBookmarkBar, + base::FeatureList::IsEnabled(features::kAppsShortcutDefaultOff) ? false + : true, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); registry->RegisterBooleanPref( prefs::kShowTabGroupsInBookmarkBar, true, diff --git a/components/bookmarks/browser/features.cc b/components/bookmarks/browser/features.cc new file mode 100644 --- /dev/null +++ b/components/bookmarks/browser/features.cc @@ -0,0 +1,17 @@ +// Copyright 2021 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#include "components/bookmarks/browser/features.h" + +namespace bookmarks { +namespace features { + +// Changes the apps shortcut on the bookmarks bar to default to off. +// https://crbug.com/1236793 +CROMITE_FEATURE(kAppsShortcutDefaultOff, + "AppsShortcutDefaultOff", + base::FEATURE_ENABLED_BY_DEFAULT); + +} // namespace features +} // namespace bookmarks diff --git a/components/bookmarks/browser/features.h b/components/bookmarks/browser/features.h new file mode 100644 --- /dev/null +++ b/components/bookmarks/browser/features.h @@ -0,0 +1,18 @@ +// Copyright 2021 The Chromium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +#ifndef COMPONENTS_BOOKMARKS_BROWSER_FEATURES_H_ +#define COMPONENTS_BOOKMARKS_BROWSER_FEATURES_H_ + +#include "base/feature_list.h" + +namespace bookmarks { +namespace features { + +BASE_DECLARE_FEATURE(kAppsShortcutDefaultOff); + +} // namespace features +} // namespace bookmarks + +#endif // COMPONENTS_BOOKMARKS_BROWSER_FEATURES_H_ diff --git a/components/content_settings/core/browser/content_settings_registry.cc b/components/content_settings/core/browser/content_settings_registry.cc --- a/components/content_settings/core/browser/content_settings_registry.cc +++ b/components/content_settings/core/browser/content_settings_registry.cc @@ -156,7 +156,7 @@ void ContentSettingsRegistry::Init() { PermissionSettingsInfo::EXCEPTIONS_ON_SECURE_ORIGINS_ONLY); Register(ContentSettingsType::NOTIFICATIONS, "notifications", - CONTENT_SETTING_ASK, WebsiteSettingsInfo::UNSYNCABLE, + CONTENT_SETTING_BLOCK, WebsiteSettingsInfo::UNSYNCABLE, /*allowlisted_primary_schemes=*/{}, /*valid_settings=*/ {CONTENT_SETTING_ALLOW, CONTENT_SETTING_BLOCK, CONTENT_SETTING_ASK}, @@ -217,7 +217,7 @@ void ContentSettingsRegistry::Init() { PermissionSettingsInfo::EXCEPTIONS_ON_SECURE_ORIGINS_ONLY); Register(ContentSettingsType::PROTECTED_MEDIA_IDENTIFIER, - "protected-media-identifier", CONTENT_SETTING_ALLOW, + "protected-media-identifier", CONTENT_SETTING_BLOCK, WebsiteSettingsInfo::UNSYNCABLE, /*allowlisted_primary_schemes=*/{}, #if BUILDFLAG(IS_ANDROID) /*valid_settings=*/ @@ -244,7 +244,7 @@ void ContentSettingsRegistry::Init() { PermissionSettingsInfo::EXCEPTIONS_ON_SECURE_ORIGINS_ONLY); Register(ContentSettingsType::BACKGROUND_SYNC, "background-sync", - CONTENT_SETTING_ALLOW, WebsiteSettingsInfo::UNSYNCABLE, + CONTENT_SETTING_BLOCK, WebsiteSettingsInfo::UNSYNCABLE, /*allowlisted_primary_schemes=*/{}, /*valid_settings=*/{CONTENT_SETTING_ALLOW, CONTENT_SETTING_BLOCK}, WebsiteSettingsInfo::TOP_ORIGIN_ONLY_SCOPE, @@ -359,7 +359,7 @@ void ContentSettingsRegistry::Init() { // TODO(crbug.com/40602007): Update this to "SECURE_ONLY" once // DeviceOrientationEvents and DeviceMotionEvents are only fired in secure // contexts. - Register(ContentSettingsType::SENSORS, "sensors", CONTENT_SETTING_ALLOW, + Register(ContentSettingsType::SENSORS, "sensors", CONTENT_SETTING_BLOCK, WebsiteSettingsInfo::UNSYNCABLE, /*allowlisted_primary_schemes=*/{}, /*valid_settings=*/ {CONTENT_SETTING_ALLOW, CONTENT_SETTING_BLOCK, CONTENT_SETTING_ASK}, diff --git a/components/content_settings/core/browser/cookie_settings.cc b/components/content_settings/core/browser/cookie_settings.cc --- a/components/content_settings/core/browser/cookie_settings.cc +++ b/components/content_settings/core/browser/cookie_settings.cc @@ -83,7 +83,7 @@ void CookieSettings::RegisterProfilePrefs( user_prefs::PrefRegistrySyncable* registry) { registry->RegisterIntegerPref( prefs::kCookieControlsMode, - static_cast(CookieControlsMode::kIncognitoOnly), + static_cast(CookieControlsMode::kBlockThirdParty), // this is the default in Bromite, overridden from kIncognitoOnly user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); } diff --git a/components/payments/core/payment_prefs.cc b/components/payments/core/payment_prefs.cc --- a/components/payments/core/payment_prefs.cc +++ b/components/payments/core/payment_prefs.cc @@ -11,7 +11,7 @@ namespace payments { void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry) { registry->RegisterBooleanPref(kPaymentsFirstTransactionCompleted, false); registry->RegisterBooleanPref( - kCanMakePaymentEnabled, true, + kCanMakePaymentEnabled, false, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); } diff --git a/components/safe_browsing/core/common/safe_browsing_prefs.cc b/components/safe_browsing/core/common/safe_browsing_prefs.cc --- a/components/safe_browsing/core/common/safe_browsing_prefs.cc +++ b/components/safe_browsing/core/common/safe_browsing_prefs.cc @@ -270,13 +270,13 @@ void RegisterProfilePrefs(PrefRegistrySimple* registry) { registry->RegisterBooleanPref( prefs::kSafeBrowsingSawInterstitialScoutReporting, false); registry->RegisterBooleanPref( - prefs::kSafeBrowsingExtendedReportingOptInAllowed, true); + prefs::kSafeBrowsingExtendedReportingOptInAllowed, false); registry->RegisterTimePref( prefs::kSafeBrowsingEsbProtegoPingWithTokenLastLogTime, base::Time()); registry->RegisterTimePref( prefs::kSafeBrowsingEsbProtegoPingWithoutTokenLastLogTime, base::Time()); registry->RegisterBooleanPref( - prefs::kSafeBrowsingEnabled, true, + prefs::kSafeBrowsingEnabled, false, user_prefs::PrefRegistrySyncable::SYNCABLE_PREF); if (base::FeatureList::IsEnabled(kEsbAsASyncedSetting)) { registry->RegisterBooleanPref( diff --git a/components/security_interstitials/core/features.cc b/components/security_interstitials/core/features.cc --- a/components/security_interstitials/core/features.cc +++ b/components/security_interstitials/core/features.cc @@ -18,6 +18,7 @@ BASE_FEATURE(kHttpsFirstDialogUi, base::FEATURE_DISABLED_BY_DEFAULT); #else BASE_FEATURE(kHttpsFirstDialogUi, base::FEATURE_ENABLED_BY_DEFAULT); #endif +SET_CROMITE_FEATURE_DISABLED(kHttpsFirstDialogUi); BASE_FEATURE(kInsecureFormNavigationThrottleForPrerender, base::FEATURE_ENABLED_BY_DEFAULT); diff --git a/components/signin/internal/identity_manager/primary_account_manager.cc b/components/signin/internal/identity_manager/primary_account_manager.cc --- a/components/signin/internal/identity_manager/primary_account_manager.cc +++ b/components/signin/internal/identity_manager/primary_account_manager.cc @@ -363,7 +363,7 @@ void PrimaryAccountManager::RegisterProfilePrefs(PrefRegistrySimple* registry) { prefs::kGoogleServicesSyncingUsernameMigratedToSignedIn, std::string()); registry->RegisterIntegerPref(prefs::kGoogleServicesSyncingUserMigrationType, /*SyncToSigninMigrationType::kUnknown=*/0); - registry->RegisterBooleanPref(prefs::kSigninAllowed, true); + registry->RegisterBooleanPref(prefs::kSigninAllowed, false); registry->RegisterBooleanPref(prefs::kSignedInWithCredentialProvider, false); registry->RegisterBooleanPref(prefs::kExplicitBrowserSignin, false); registry->RegisterBooleanPref( diff --git a/cromite_flags/chrome/browser/about_flags_cc/Modify-default-preferences.inc b/cromite_flags/chrome/browser/about_flags_cc/Modify-default-preferences.inc new file mode 100755 --- /dev/null +++ b/cromite_flags/chrome/browser/about_flags_cc/Modify-default-preferences.inc @@ -0,0 +1,8 @@ +#ifdef FLAG_SECTION + + {"apps-shortcut-default-off", + "Apps Shortcut Default Off", + "Changes the apps shortcut on the bookmarks bar to default to off.", kOsDesktop, + FEATURE_VALUE_TYPE(bookmarks::features::kAppsShortcutDefaultOff)}, + +#endif diff --git a/cromite_flags/third_party/blink/common/features_cc/Modify-default-preferences.inc b/cromite_flags/third_party/blink/common/features_cc/Modify-default-preferences.inc new file mode 100755 --- /dev/null +++ b/cromite_flags/third_party/blink/common/features_cc/Modify-default-preferences.inc @@ -0,0 +1 @@ +SET_CROMITE_FEATURE_ENABLED(kIsolateSandboxedIframes); --