I think the added indentation on `this.app...`
````
        if ((modifiers & Clutter.ModifierType.CONTROL_MASK &&
                    this.app.state == Shell.AppState.RUNNING) ||
            !this._isAppOnActiveWorkspace()) {
````
will give more readability that there's a preceeding `(modifiers & Clutter.ModifierType.CONTROL_MASK && this.app.state == Shell.AppState.RUNNING)` before conditional or with `!this._isAppOnActiveWorkspace()`, or do you prefer this?
````
        if (!this._isAppOnActiveWorkspace() ||
            (modifiers & Clutter.ModifierType.CONTROL_MASK &&
                    this.app.state == Shell.AppState.RUNNING)) {
````