2017-12-05 14:05:00 -05:00
|
|
|
// -*- mode: js; js-indent-level: 4; indent-tabs-mode: nil -*-
|
2019-01-31 09:07:06 -05:00
|
|
|
/* exported InputMethod */
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
const { Clutter, GLib, Gio, GObject, IBus } = imports.gi;
|
2019-02-08 22:21:36 -05:00
|
|
|
|
2017-12-05 14:05:00 -05:00
|
|
|
const Keyboard = imports.ui.status.keyboard;
|
|
|
|
|
2019-12-19 14:50:37 -05:00
|
|
|
Gio._promisify(IBus.Bus.prototype,
|
|
|
|
'create_input_context_async', 'create_input_context_async_finish');
|
|
|
|
|
2019-02-08 17:00:18 -05:00
|
|
|
var HIDE_PANEL_TIME = 50;
|
|
|
|
|
2017-10-30 21:23:39 -04:00
|
|
|
var InputMethod = GObject.registerClass(
|
|
|
|
class InputMethod extends Clutter.InputMethod {
|
2017-10-30 20:03:21 -04:00
|
|
|
_init() {
|
2017-10-30 21:23:39 -04:00
|
|
|
super._init();
|
2017-12-05 14:05:00 -05:00
|
|
|
this._hints = 0;
|
|
|
|
this._purpose = 0;
|
|
|
|
this._currentFocus = null;
|
2018-08-21 07:21:53 -04:00
|
|
|
this._preeditStr = '';
|
|
|
|
this._preeditPos = 0;
|
2018-11-13 12:28:15 -05:00
|
|
|
this._preeditVisible = false;
|
2019-02-08 17:00:18 -05:00
|
|
|
this._hidePanelId = 0;
|
2017-12-05 14:05:00 -05:00
|
|
|
this._ibus = IBus.Bus.new_async();
|
2017-12-01 19:27:35 -05:00
|
|
|
this._ibus.connect('connected', this._onConnected.bind(this));
|
|
|
|
this._ibus.connect('disconnected', this._clear.bind(this));
|
|
|
|
this.connect('notify::can-show-preedit', this._updateCapabilities.bind(this));
|
2017-12-05 14:05:00 -05:00
|
|
|
|
|
|
|
this._inputSourceManager = Keyboard.getInputSourceManager();
|
|
|
|
this._sourceChangedId = this._inputSourceManager.connect('current-source-changed',
|
2017-12-01 19:27:35 -05:00
|
|
|
this._onSourceChanged.bind(this));
|
2017-12-05 14:05:00 -05:00
|
|
|
this._currentSource = this._inputSourceManager.currentSource;
|
|
|
|
|
|
|
|
if (this._ibus.is_connected())
|
|
|
|
this._onConnected();
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
|
|
|
get currentFocus() {
|
|
|
|
return this._currentFocus;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
_updateCapabilities() {
|
2019-10-01 06:53:20 -04:00
|
|
|
let caps = IBus.Capabilite.PREEDIT_TEXT | IBus.Capabilite.FOCUS | IBus.Capabilite.SURROUNDING_TEXT;
|
2017-12-05 14:05:00 -05:00
|
|
|
|
|
|
|
if (this._context)
|
|
|
|
this._context.set_capabilities(caps);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
_onSourceChanged() {
|
2017-12-05 14:05:00 -05:00
|
|
|
this._currentSource = this._inputSourceManager.currentSource;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2019-12-19 14:50:37 -05:00
|
|
|
async _onConnected() {
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
this._cancellable = new Gio.Cancellable();
|
|
|
|
try {
|
2019-12-19 14:50:37 -05:00
|
|
|
this._context = await this._ibus.create_input_context_async(
|
|
|
|
'gnome-shell', -1, this._cancellable);
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
} catch (e) {
|
|
|
|
if (!e.matches(Gio.IOErrorEnum, Gio.IOErrorEnum.CANCELLED)) {
|
|
|
|
logError(e);
|
|
|
|
this._clear();
|
|
|
|
}
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2021-07-27 10:52:32 -04:00
|
|
|
this._context.set_client_commit_preedit(true);
|
2017-12-01 19:27:35 -05:00
|
|
|
this._context.connect('commit-text', this._onCommitText.bind(this));
|
|
|
|
this._context.connect('delete-surrounding-text', this._onDeleteSurroundingText.bind(this));
|
2021-07-27 10:52:32 -04:00
|
|
|
this._context.connect('update-preedit-text-with-mode', this._onUpdatePreeditText.bind(this));
|
2018-08-21 07:21:53 -04:00
|
|
|
this._context.connect('show-preedit-text', this._onShowPreeditText.bind(this));
|
|
|
|
this._context.connect('hide-preedit-text', this._onHidePreeditText.bind(this));
|
2018-06-29 11:35:39 -04:00
|
|
|
this._context.connect('forward-key-event', this._onForwardKeyEvent.bind(this));
|
2019-11-07 13:06:15 -05:00
|
|
|
this._context.connect('destroy', this._clear.bind(this));
|
2017-12-05 14:05:00 -05:00
|
|
|
|
|
|
|
this._updateCapabilities();
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
_clear() {
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
if (this._cancellable) {
|
|
|
|
this._cancellable.cancel();
|
|
|
|
this._cancellable = null;
|
|
|
|
}
|
|
|
|
|
2017-12-05 14:05:00 -05:00
|
|
|
this._context = null;
|
|
|
|
this._hints = 0;
|
|
|
|
this._purpose = 0;
|
2019-01-28 20:18:52 -05:00
|
|
|
this._preeditStr = '';
|
2018-08-21 07:21:53 -04:00
|
|
|
this._preeditPos = 0;
|
2018-11-13 12:28:15 -05:00
|
|
|
this._preeditVisible = false;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
_emitRequestSurrounding() {
|
2017-12-05 14:05:00 -05:00
|
|
|
if (this._context.needs_surrounding_text())
|
|
|
|
this.emit('request-surrounding');
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2019-01-31 09:08:10 -05:00
|
|
|
_onCommitText(_context, text) {
|
2017-12-05 14:05:00 -05:00
|
|
|
this.commit(text.get_text());
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2019-08-14 04:16:46 -04:00
|
|
|
_onDeleteSurroundingText(_context, offset, nchars) {
|
2020-03-28 12:26:39 -04:00
|
|
|
try {
|
|
|
|
this.delete_surrounding(offset, nchars);
|
|
|
|
} catch (e) {
|
|
|
|
// We may get out of bounds for negative offset on older mutter
|
|
|
|
this.delete_surrounding(0, nchars + offset);
|
|
|
|
}
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2021-07-27 10:52:32 -04:00
|
|
|
_onUpdatePreeditText(_context, text, pos, visible, mode) {
|
2018-08-21 07:21:53 -04:00
|
|
|
if (text == null)
|
|
|
|
return;
|
2018-11-13 12:28:15 -05:00
|
|
|
|
|
|
|
let preedit = text.get_text();
|
|
|
|
|
2018-08-21 07:21:53 -04:00
|
|
|
if (visible)
|
2021-07-27 10:52:32 -04:00
|
|
|
this.set_preedit_text(preedit, pos, mode);
|
2018-11-13 12:28:15 -05:00
|
|
|
else if (this._preeditVisible)
|
2021-07-27 10:52:32 -04:00
|
|
|
this.set_preedit_text(null, pos, mode);
|
2018-11-13 12:28:15 -05:00
|
|
|
|
|
|
|
this._preeditStr = preedit;
|
|
|
|
this._preeditPos = pos;
|
|
|
|
this._preeditVisible = visible;
|
2021-07-27 10:52:32 -04:00
|
|
|
this._preeditCommitMode = mode;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2018-08-21 07:21:53 -04:00
|
|
|
|
2019-02-04 06:30:53 -05:00
|
|
|
_onShowPreeditText() {
|
2018-11-13 12:28:15 -05:00
|
|
|
this._preeditVisible = true;
|
2021-07-27 10:52:32 -04:00
|
|
|
this.set_preedit_text(this._preeditStr, this._preeditPos, this._preeditCommitMode);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2019-02-04 06:30:53 -05:00
|
|
|
_onHidePreeditText() {
|
2021-07-27 10:52:32 -04:00
|
|
|
this.set_preedit_text(null, this._preeditPos, this._preeditCommitMode);
|
2018-11-13 12:28:15 -05:00
|
|
|
this._preeditVisible = false;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2019-01-31 09:08:10 -05:00
|
|
|
_onForwardKeyEvent(_context, keyval, keycode, state) {
|
2018-06-29 11:35:39 -04:00
|
|
|
let press = (state & IBus.ModifierType.RELEASE_MASK) == 0;
|
2019-08-19 15:38:51 -04:00
|
|
|
state &= ~IBus.ModifierType.RELEASE_MASK;
|
2018-06-29 11:35:39 -04:00
|
|
|
|
2018-09-27 15:09:02 -04:00
|
|
|
let curEvent = Clutter.get_current_event();
|
|
|
|
let time;
|
|
|
|
if (curEvent)
|
|
|
|
time = curEvent.get_time();
|
|
|
|
else
|
|
|
|
time = global.display.get_current_time_roundtrip();
|
2018-06-29 11:35:39 -04:00
|
|
|
|
2018-09-27 15:09:02 -04:00
|
|
|
this.forward_key(keyval, keycode + 8, state & Clutter.ModifierType.MODIFIER_MASK, time, press);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2018-06-29 11:35:39 -04:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_focus_in(focus) {
|
2017-12-05 14:05:00 -05:00
|
|
|
this._currentFocus = focus;
|
|
|
|
if (this._context) {
|
|
|
|
this._context.focus_in();
|
|
|
|
this._emitRequestSurrounding();
|
|
|
|
}
|
2019-02-08 17:00:18 -05:00
|
|
|
|
|
|
|
if (this._hidePanelId) {
|
|
|
|
GLib.source_remove(this._hidePanelId);
|
|
|
|
this._hidePanelId = 0;
|
|
|
|
}
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_focus_out() {
|
2017-12-05 14:05:00 -05:00
|
|
|
this._currentFocus = null;
|
2019-10-01 06:53:20 -04:00
|
|
|
if (this._context)
|
2017-12-05 14:05:00 -05:00
|
|
|
this._context.focus_out();
|
|
|
|
|
2018-11-13 12:26:13 -05:00
|
|
|
if (this._preeditStr) {
|
|
|
|
// Unset any preedit text
|
2021-07-27 10:52:32 -04:00
|
|
|
this.set_preedit_text(null, 0, this._preeditCommitMode);
|
2018-11-13 12:26:13 -05:00
|
|
|
this._preeditStr = null;
|
|
|
|
}
|
2019-02-08 17:00:18 -05:00
|
|
|
|
|
|
|
this._hidePanelId = GLib.timeout_add(GLib.PRIORITY_DEFAULT, HIDE_PANEL_TIME, () => {
|
|
|
|
this.set_input_panel_state(Clutter.InputPanelState.OFF);
|
|
|
|
this._hidePanelId = 0;
|
|
|
|
return GLib.SOURCE_REMOVE;
|
|
|
|
});
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_reset() {
|
2021-09-29 15:46:45 -04:00
|
|
|
if (this._context) {
|
|
|
|
this._context.reset();
|
|
|
|
this._emitRequestSurrounding();
|
|
|
|
}
|
|
|
|
|
2021-07-27 10:52:32 -04:00
|
|
|
if (this._preeditStr !== null) {
|
|
|
|
// Unset any preedit text
|
|
|
|
this.set_preedit_text(null, 0, Clutter.PreeditResetMode.CLEAR);
|
|
|
|
this._preeditStr = null;
|
|
|
|
}
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_set_cursor_location(rect) {
|
2017-12-05 14:05:00 -05:00
|
|
|
if (this._context) {
|
|
|
|
this._context.set_cursor_location(rect.get_x(), rect.get_y(),
|
|
|
|
rect.get_width(), rect.get_height());
|
|
|
|
this._emitRequestSurrounding();
|
|
|
|
}
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_set_surrounding(text, cursor, anchor) {
|
2018-09-11 09:36:35 -04:00
|
|
|
if (!this._context || !text)
|
|
|
|
return;
|
|
|
|
|
|
|
|
let ibusText = IBus.Text.new_from_string(text);
|
|
|
|
this._context.set_surrounding_text(ibusText, cursor, anchor);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_update_content_hints(hints) {
|
2017-12-05 14:05:00 -05:00
|
|
|
let ibusHints = 0;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.COMPLETION)
|
|
|
|
ibusHints |= IBus.InputHints.WORD_COMPLETION;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.SPELLCHECK)
|
|
|
|
ibusHints |= IBus.InputHints.SPELLCHECK;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.AUTO_CAPITALIZATION)
|
|
|
|
ibusHints |= IBus.InputHints.UPPERCASE_SENTENCES;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.LOWERCASE)
|
|
|
|
ibusHints |= IBus.InputHints.LOWERCASE;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.UPPERCASE)
|
|
|
|
ibusHints |= IBus.InputHints.UPPERCASE_CHARS;
|
|
|
|
if (hints & Clutter.InputContentHintFlags.TITLECASE)
|
|
|
|
ibusHints |= IBus.InputHints.UPPERCASE_WORDS;
|
|
|
|
|
|
|
|
this._hints = ibusHints;
|
|
|
|
if (this._context)
|
|
|
|
this._context.set_content_type(this._purpose, this._hints);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_update_content_purpose(purpose) {
|
2017-12-05 14:05:00 -05:00
|
|
|
let ibusPurpose = 0;
|
|
|
|
if (purpose == Clutter.InputContentPurpose.NORMAL)
|
|
|
|
ibusPurpose = IBus.InputPurpose.FREE_FORM;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.ALPHA)
|
|
|
|
ibusPurpose = IBus.InputPurpose.ALPHA;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.DIGITS)
|
|
|
|
ibusPurpose = IBus.InputPurpose.DIGITS;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.NUMBER)
|
|
|
|
ibusPurpose = IBus.InputPurpose.NUMBER;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.PHONE)
|
|
|
|
ibusPurpose = IBus.InputPurpose.PHONE;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.URL)
|
|
|
|
ibusPurpose = IBus.InputPurpose.URL;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.EMAIL)
|
|
|
|
ibusPurpose = IBus.InputPurpose.EMAIL;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.NAME)
|
|
|
|
ibusPurpose = IBus.InputPurpose.NAME;
|
|
|
|
else if (purpose == Clutter.InputContentPurpose.PASSWORD)
|
|
|
|
ibusPurpose = IBus.InputPurpose.PASSWORD;
|
|
|
|
|
|
|
|
this._purpose = ibusPurpose;
|
|
|
|
if (this._context)
|
|
|
|
this._context.set_content_type(this._purpose, this._hints);
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
|
2017-10-30 20:03:21 -04:00
|
|
|
vfunc_filter_key_event(event) {
|
2018-12-11 02:18:59 -05:00
|
|
|
if (!this._context)
|
2017-12-05 14:05:00 -05:00
|
|
|
return false;
|
2018-03-15 09:01:51 -04:00
|
|
|
if (!this._currentSource)
|
2017-12-05 14:05:00 -05:00
|
|
|
return false;
|
|
|
|
|
|
|
|
let state = event.get_state();
|
|
|
|
if (state & IBus.ModifierType.IGNORED_MASK)
|
|
|
|
return false;
|
|
|
|
|
|
|
|
if (event.type() == Clutter.EventType.KEY_RELEASE)
|
|
|
|
state |= IBus.ModifierType.RELEASE_MASK;
|
2018-06-29 11:35:39 -04:00
|
|
|
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
this._context.process_key_event_async(
|
|
|
|
event.get_key_symbol(),
|
|
|
|
event.get_key_code() - 8, // Convert XKB keycodes to evcodes
|
|
|
|
state, -1, this._cancellable,
|
|
|
|
(context, res) => {
|
2019-11-07 13:06:15 -05:00
|
|
|
if (context != this._context)
|
|
|
|
return;
|
|
|
|
|
ibusManager, inputMethod: Cancel async ibus calls chain on disconnect
The shell tries to spawn the ibus daemon on startup if unavailable, however
as per commit 8adfc5b1 we also force restarting it once the X11 server is
available.
Unfortunately this could cause a race if we disconnect while we were already
connected to an ibus daemon, but still in the process of going through the
various nested calls.
In fact the ::disconnect callback didn't stop any further async ibus call
that, even if failing, would have eventually triggered the emission of a
'ready' signal and to the Keyboard's callback, leading under X11 to a full
grab owned by ibus daemon.
In order to avoid this and keep control of the calls order, use in both
IbusManager and InputMethod a cancellable that is setup before connecting to
the bus, and that is cancelled on disconnection.
Then handle the finish() calls properly, using try/catch to validate the
returned value, taking in account the potential error and just not
proceeding in case of cancellation.
Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1712
2019-09-25 14:56:46 -04:00
|
|
|
try {
|
|
|
|
let retval = context.process_key_event_async_finish(res);
|
|
|
|
this.notify_key_event(event, retval);
|
|
|
|
} catch (e) {
|
|
|
|
if (!e.matches(Gio.IOErrorEnum, Gio.IOErrorEnum.CANCELLED))
|
|
|
|
log(`Error processing key on IM: ${e.message}`);
|
|
|
|
}
|
|
|
|
});
|
2017-12-05 14:05:00 -05:00
|
|
|
return true;
|
2017-10-30 21:23:39 -04:00
|
|
|
}
|
2017-12-05 14:05:00 -05:00
|
|
|
});
|