Commit ad40d624 authored by Kyle Milka's avatar Kyle Milka Committed by Commit Bot

[NTP] Remove HideShortcutsOnNtp

Not launching.

Bug: None
Change-Id: I546fa7cabbbc1909eb1d690ffae8cef56f8a31eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759185Reviewed-by: default avatarKristi Park <kristipark@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688341}
parent 96871e6c
......@@ -322,8 +322,6 @@ customize.setMenuVisibility = function() {
$(customize.IDS.UPLOAD_IMAGE).hidden = false;
$(customize.IDS.RESTORE_DEFAULT).hidden = false;
$(customize.IDS.EDIT_BG_DIVIDER).hidden = false;
$(customize.IDS.CUSTOM_LINKS_RESTORE_DEFAULT).hidden =
configData.hideShortcuts;
$(customize.IDS.COLORS_BUTTON).hidden = !configData.chromeColors;
$(customize.IDS.COLOR_PICKER_CONTAINER)
.classList.toggle(
......
......@@ -29,7 +29,6 @@ let MostVisitedData;
* @typedef {{chromeColors: boolean,
* enableShortcutsGrid: boolean,
* googleBaseUrl: string,
* hideShortcuts: boolean,
* isAccessibleBrowser: boolean,
* isGooglePage: boolean,
* richerPicker: boolean,
......
......@@ -922,7 +922,7 @@ function handlePostMessage(event) {
if ($(IDS.PROMO)) {
$(IDS.PROMO).classList.add(CLASSES.SHOW_ELEMENT);
}
if (customLinksEnabled() && !configData.hideShortcuts) {
if (customLinksEnabled()) {
$(customize.IDS.CUSTOM_LINKS_RESTORE_DEFAULT)
.classList.toggle(
customize.CLASSES.OPTION_DISABLED, !args.showRestoreDefault);
......@@ -1243,11 +1243,6 @@ function createIframes() {
clIframeDialog.classList.add(CLASSES.CUSTOMIZE_DIALOG);
clIframeDialog.appendChild(clIframe);
document.body.appendChild(clIframeDialog);
if (configData.hideShortcuts) {
$(IDS.TILES).style.display = 'none';
clIframeDialog.style.display = 'none';
}
}
window.addEventListener('message', handlePostMessage);
......
......@@ -604,9 +604,6 @@ class LocalNtpSource::SearchConfigurationProvider
config_data.SetBoolean(
"enableShortcutsGrid",
base::FeatureList::IsEnabled(features::kGridLayoutForNtpShortcuts));
config_data.SetBoolean(
"hideShortcuts",
base::FeatureList::IsEnabled(features::kHideShortcutsOnNtp));
config_data.SetBoolean(
"showFakeboxPlaceholderOnFocus",
base::FeatureList::IsEnabled(
......
......@@ -35,8 +35,4 @@ const base::Feature kNtpCustomizationMenuV2{"NtpCustomizationMenuV2",
const base::Feature kFirstRunDefaultSearchShortcut{
"FirstRunDefaultSearchShortcut", base::FEATURE_DISABLED_BY_DEFAULT};
// If enabled, the shortcuts will not be shown on the NTP.
const base::Feature kHideShortcutsOnNtp{"HideShortcutsOnNtp",
base::FEATURE_DISABLED_BY_DEFAULT};
} // namespace features
......@@ -19,7 +19,6 @@ extern const base::Feature kGridLayoutForNtpShortcuts;
extern const base::Feature kNtpCustomizationMenuV2;
extern const base::Feature kFirstRunDefaultSearchShortcut;
extern const base::Feature kHideShortcutsOnNtp;
} // namespace features
......
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