Skip to content

Commit 3ff22c9

Browse files
authored
Merge pull request #10314 from brave/pr10299_maxk-hide-idle-detection-ui_1.31.x
Hide idle detection UI elements in Settings. (uplift to 1.31.x)
2 parents ba88288 + 02e7f75 commit 3ff22c9

File tree

4 files changed

+31
-0
lines changed

4 files changed

+31
-0
lines changed

browser/resources/settings/brave_overrides/privacy_page.js

+9
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,22 @@
55

66
import {RegisterPolymerTemplateModifications} from 'chrome://brave-resources/polymer_overriding.js'
77
import {I18nBehavior} from 'chrome://resources/js/i18n_behavior.m.js'
8+
import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
89

910
RegisterPolymerTemplateModifications({
1011
'settings-privacy-page': (templateContent) => {
1112
const pages = templateContent.getElementById('pages')
1213
if (!pages) {
1314
console.error(`[Brave Settings Overrides] Couldn't find privacy_page #pages`)
1415
} else {
16+
if (!loadTimeData.getBoolean('isIdleDetectionFeatureEnabled')) {
17+
const idleDetection = templateContent.querySelector('[route-path="/content/idleDetection"]')
18+
if (!idleDetection) {
19+
console.error(`[Brave Settings Overrides] Couldn't find idle detection template`)
20+
} else {
21+
idleDetection.content.firstElementChild.hidden = true
22+
}
23+
}
1524
pages.insertAdjacentHTML('beforeend', `
1625
<template is="dom-if" route-path="/content/autoplay" no-search>
1726
<settings-subpage page-title="${I18nBehavior.i18n('siteSettingsCategoryAutoplay')}">

browser/resources/settings/brave_overrides/site_details.js

+9
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,18 @@
55

66
import {RegisterPolymerTemplateModifications} from 'chrome://brave-resources/polymer_overriding.js'
77
import {I18nBehavior} from 'chrome://resources/js/i18n_behavior.m.js'
8+
import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
89

910
RegisterPolymerTemplateModifications({
1011
'site-details': (templateContent) => {
12+
if (!loadTimeData.getBoolean('isIdleDetectionFeatureEnabled')) {
13+
const idleDetectionItem = templateContent.querySelector('[category="[[contentSettingsTypesEnum_.IDLE_DETECTION]]"]')
14+
if (!idleDetectionItem) {
15+
console.error(`[Brave Settings Overrides] Couldn't find idle detection item`)
16+
} else {
17+
idleDetectionItem.hidden = true
18+
}
19+
}
1120
const firstPermissionItem = templateContent.querySelector('div.list-frame > site-details-permission:nth-child(1)')
1221
if (!firstPermissionItem) {
1322
console.error(`[Brave Settings Overrides] Couldn't find first permission item`)

browser/resources/settings/brave_overrides/site_settings_page.js

+9
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
import {define, RegisterPolymerComponentReplacement} from 'chrome://brave-resources/polymer_overriding.js'
77
import {ContentSettingsTypes} from '../site_settings/constants.js'
8+
import {loadTimeData} from 'chrome://resources/js/load_time_data.m.js';
89
import {SettingsSiteSettingsPageElement} from '../site_settings_page/site_settings_page.js'
910
import {routes} from '../route.js'
1011
import './config.js'
@@ -45,6 +46,14 @@ RegisterPolymerComponentReplacement(
4546
if (!lists_.permissionsAdvanced) {
4647
console.error('[Brave Settings Overrides] did not get lists_.permissionsAdvanced data')
4748
} else {
49+
if (!loadTimeData.getBoolean('isIdleDetectionFeatureEnabled')) {
50+
let indexForIdleDetection = lists_.permissionsAdvanced.findIndex(item => item.id === ContentSettingsTypes.IDLE_DETECTION)
51+
if (indexForIdleDetection === -1) {
52+
console.error('Could not find idle detection site settings item')
53+
} else {
54+
lists_.permissionsAdvanced.splice(indexForIdleDetection, 1)
55+
}
56+
}
4857
let indexForAutoplay = lists_.permissionsAdvanced.findIndex(item => item.id === ContentSettingsTypes.AUTOMATIC_DOWNLOADS)
4958
if (indexForAutoplay === -1) {
5059
console.error('Could not find automatic downloads site settings item')

browser/ui/webui/brave_settings_ui.cc

+4
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#include "chrome/browser/ui/webui/settings/metrics_reporting_handler.h"
2929
#include "components/sync/driver/sync_driver_switches.h"
3030
#include "content/public/browser/web_ui_data_source.h"
31+
#include "content/public/common/content_features.h"
3132

3233
#if BUILDFLAG(ENABLE_SPARKLE)
3334
#include "brave/browser/ui/webui/settings/brave_relaunch_handler_mac.h"
@@ -84,6 +85,9 @@ void BraveSettingsUI::AddResources(content::WebUIDataSource* html_source,
8485
NavigationBarDataProvider::Initialize(html_source);
8586
if (auto* service = ViewCounterServiceFactory::GetForProfile(profile))
8687
service->InitializeWebUIDataSource(html_source);
88+
html_source->AddBoolean(
89+
"isIdleDetectionFeatureEnabled",
90+
base::FeatureList::IsEnabled(features::kIdleDetection));
8791
#if BUILDFLAG(ENABLE_SIDEBAR)
8892
// TODO(simonhong): Remove this when sidebar is shipped by default in all
8993
// channels.

0 commit comments

Comments
 (0)