From e7cf6d348c5424692c951c19b7cf1765a8234757 Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net>
Date: Thu, 15 Sep 2022 18:05:58 +0200
Subject: [PATCH] bluetooth: Remove unused 'had-bluetooth-devices-setup'
 setting

Now that we always show the Bluetooth switch when there's a Bluetooth
adapter hiding somewhere, remove all the settings it used as storage.

If the user wants to really really disable any Bluetooth adapters in
their system, the gnome-bluetooth documentation explains how to do that
irrespective of GNOME:
https://gitlab.gnome.org/GNOME/gnome-bluetooth/#multiple-bluetooth-adapters

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2488>
---
 data/org.gnome.shell.gschema.xml.in | 10 ----------
 js/ui/status/bluetooth.js           | 21 ---------------------
 2 files changed, 31 deletions(-)

diff --git a/data/org.gnome.shell.gschema.xml.in b/data/org.gnome.shell.gschema.xml.in
index c56a3399e..86ad1b24b 100644
--- a/data/org.gnome.shell.gschema.xml.in
+++ b/data/org.gnome.shell.gschema.xml.in
@@ -84,16 +84,6 @@
         This key sets the default state of the checkbox.
       </description>
     </key>
-    <key name="had-bluetooth-devices-setup" type="b">
-      <default>false</default>
-      <summary>Whether the default Bluetooth adapter had set up devices associated to it</summary>
-      <description>
-        The shell will only show a Bluetooth menu item if a Bluetooth
-        adapter is powered, or if there were devices set up associated
-        with the default adapter. This will be reset if the default
-        adapter is ever seen not to have devices associated to it.
-      </description>
-    </key>
     <key name="last-selected-power-profile" type="s">
       <default>"power-saver"</default>
       <summary>The last selected non-default power profile</summary>
diff --git a/js/ui/status/bluetooth.js b/js/ui/status/bluetooth.js
index f5ed1775b..bbff62da7 100644
--- a/js/ui/status/bluetooth.js
+++ b/js/ui/status/bluetooth.js
@@ -15,8 +15,6 @@ const OBJECT_PATH = '/org/gnome/SettingsDaemon/Rfkill';
 const RfkillManagerInterface = loadInterfaceXML('org.gnome.SettingsDaemon.Rfkill');
 const rfkillManagerInfo = Gio.DBusInterfaceInfo.new_for_xml(RfkillManagerInterface);
 
-const HAD_BLUETOOTH_DEVICES_SETUP = 'had-bluetooth-devices-setup';
-
 const BtClient = GObject.registerClass({
     Properties: {
         'available': GObject.ParamSpec.boolean('available', '', '',
@@ -36,8 +34,6 @@ const BtClient = GObject.registerClass({
     _init() {
         super._init();
 
-        this._hadSetupDevices = global.settings.get_boolean(HAD_BLUETOOTH_DEVICES_SETUP);
-
         this._client = new GnomeBluetooth.Client();
         this._client.connect('notify::default-adapter-powered', () => {
             this.notify('active');
@@ -128,29 +124,12 @@ const BtClient = GObject.registerClass({
         if (this._devicesChangedId)
             return;
         this._devicesChangedId = GLib.idle_add(GLib.PRIORITY_DEFAULT, () => {
-            this._syncHadSetupDevices();
             delete this._devicesChangedId;
             this.emit('devices-changed');
             return GLib.SOURCE_REMOVE;
         });
     }
 
-    _syncHadSetupDevices() {
-        const {defaultAdapter} = this._client;
-        if (!defaultAdapter || !this._adapter)
-            return; // ignore changes while powering up/down
-
-        const [firstDevice] = this.getDevices();
-        const hadSetupDevices = !!firstDevice;
-
-        if (this._hadSetupDevices === hadSetupDevices)
-            return;
-
-        this._hadSetupDevices = hadSetupDevices;
-        global.settings.set_boolean(
-            HAD_BLUETOOTH_DEVICES_SETUP, this._hadSetupDevices);
-    }
-
     _connectDeviceNotify(device) {
         const path = device.get_object_path();