From 8ecff981048c0a746ebce8c945ed6ece2c4ae1fc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Wed, 3 Jun 2020 02:26:44 +0200 Subject: [PATCH] Remove obsolete patches --- ...Disable-extension-before-unloading-s.patch | 51 ------------------- ...-the-panel-after-porting-to-Promises.patch | 49 ------------------ 2 files changed, 100 deletions(-) delete mode 100644 0001-extensionSystem-Disable-extension-before-unloading-s.patch delete mode 100644 0001-ibusManager-Fix-the-panel-after-porting-to-Promises.patch diff --git a/0001-extensionSystem-Disable-extension-before-unloading-s.patch b/0001-extensionSystem-Disable-extension-before-unloading-s.patch deleted file mode 100644 index 9fc414e..0000000 --- a/0001-extensionSystem-Disable-extension-before-unloading-s.patch +++ /dev/null @@ -1,51 +0,0 @@ -From 6d5e93b00b5b4ce5315276e071a98c8db5ff6463 Mon Sep 17 00:00:00 2001 -From: =?UTF-8?q?Florian=20M=C3=BCllner?= -Date: Wed, 13 May 2020 20:10:56 +0200 -Subject: [PATCH] extensionSystem: Disable extension before unloading - stylesheet - -Removing a stylesheet from the theme will trigger a style update. There's -little point in updating the extension actors that are about to be destroyed -(hopefully), so call the extension's disable() function first. - -https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/2757 ---- - js/ui/extensionSystem.js | 12 ++++++------ - 1 file changed, 6 insertions(+), 6 deletions(-) - -diff --git a/js/ui/extensionSystem.js b/js/ui/extensionSystem.js -index 1a34f19e9e384b1d352207ea78eba9e7e1f80ba3..5a50b473ac2bdb25ec1a7f47cbd2d00ebf889800 100644 ---- a/js/ui/extensionSystem.js -+++ b/js/ui/extensionSystem.js -@@ -100,22 +100,22 @@ var ExtensionManager = class { - } catch (e) { - this.logExtensionError(otherUuid, e); - } - } - -+ try { -+ extension.stateObj.disable(); -+ } catch (e) { -+ this.logExtensionError(uuid, e); -+ } -+ - if (extension.stylesheet) { - let theme = St.ThemeContext.get_for_stage(global.stage).get_theme(); - theme.unload_stylesheet(extension.stylesheet); - delete extension.stylesheet; - } - -- try { -- extension.stateObj.disable(); -- } catch (e) { -- this.logExtensionError(uuid, e); -- } -- - for (let i = 0; i < order.length; i++) { - let otherUuid = order[i]; - try { - this.lookup(otherUuid).stateObj.enable(); - } catch (e) { --- -2.26.2 - diff --git a/0001-ibusManager-Fix-the-panel-after-porting-to-Promises.patch b/0001-ibusManager-Fix-the-panel-after-porting-to-Promises.patch deleted file mode 100644 index 898d90b..0000000 --- a/0001-ibusManager-Fix-the-panel-after-porting-to-Promises.patch +++ /dev/null @@ -1,49 +0,0 @@ -From e08a4acd06d59cb94585e291356ec591d4779910 Mon Sep 17 00:00:00 2001 -From: Ting-Wei Lan -Date: Sun, 3 May 2020 23:26:39 +0800 -Subject: [PATCH] ibusManager: Fix the panel after porting to Promises - -Commit 764527c8c9b7659901eb6296a6859ae2b0eabdb8 not only ports this file -to Promises but also changes the behavior of _initPanelService method. -Instead of always calling _updateReadiness when _panelService is ready, -it only calls it when get_global_engine_async succeeds. - -The only callers of _updateReadiness are _initEngines and -_initPanelService. Assume that _initEngines completes first. Its -_updateReadiness call keeps _ready as false and it is expected for -_initPanelService to change it to true. However, since -get_global_engine_async fails because there is no active engine, -_initPanelService never calls _updateReadiness. Therefore, all setEngine -calls do nothing because _ready is false, and the input method panel -never shows. Users are unable to use any input method even if they can -see that ibus-daemon is already running. - -Fix the issue by changing it back to the old behavior. - -https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1235 ---- - js/misc/ibusManager.js | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/js/misc/ibusManager.js b/js/misc/ibusManager.js -index e8cfead2c..d9a9a9d60 100644 ---- a/js/misc/ibusManager.js -+++ b/js/misc/ibusManager.js -@@ -171,13 +171,13 @@ var IBusManager = class { - this._panelService.connect('set-content-type', this._setContentType.bind(this)); - } catch (e) { - } -+ this._updateReadiness(); - - try { - // If an engine is already active we need to get its properties - const engine = - await this._ibus.get_global_engine_async(-1, this._cancellable); - this._engineChanged(this._ibus, engine.get_name()); -- this._updateReadiness(); - } catch (e) { - } - } --- -2.26.2 -