From dd708199ea2782b6ee05a7fb0ec041424a7c3a7b Mon Sep 17 00:00:00 2001 From: Orestis Floros Date: Fri, 14 Dec 2018 20:37:29 +0200 Subject: [PATCH] Fix: killing unfocused window shouldn't produce focus event --- src/x.c | 7 ++++++- testcases/t/219-ipc-window-focus.t | 15 +++++++++++++++ 2 files changed, 21 insertions(+), 1 deletion(-) diff --git a/src/x.c b/src/x.c index 82c19d30..a2f2c39f 100644 --- a/src/x.c +++ b/src/x.c @@ -276,7 +276,12 @@ static void _x_con_kill(Con *con) { free(state); /* Invalidate focused_id to correctly focus new windows with the same ID */ - focused_id = last_focused = XCB_NONE; + if (con->frame.id == focused_id) { + focused_id = XCB_NONE; + } + if (con->frame.id == last_focused) { + last_focused = XCB_NONE; + } } /* diff --git a/testcases/t/219-ipc-window-focus.t b/testcases/t/219-ipc-window-focus.t index b1c8ba18..afb1bfbc 100644 --- a/testcases/t/219-ipc-window-focus.t +++ b/testcases/t/219-ipc-window-focus.t @@ -44,11 +44,26 @@ sub focus_subtest { is($events[0]->{container}->{name}, $name, "$name focused"); } +sub kill_subtest { + my ($cmd, $name) = @_; + + my $focus = AnyEvent->condvar; + + my @events = events_for( + sub { cmd $cmd }, + 'window'); + + is(scalar @events, 1, 'Received 1 event'); + is($events[0]->{change}, 'close', 'Close event received'); + is($events[0]->{container}->{name}, $name, "$name closed"); +} + subtest 'focus left (1)', \&focus_subtest, 'focus left', $win1->name; subtest 'focus left (2)', \&focus_subtest, 'focus left', $win0->name; subtest 'focus right (1)', \&focus_subtest, 'focus right', $win1->name; subtest 'focus right (2)', \&focus_subtest, 'focus right', $win2->name; subtest 'focus right (3)', \&focus_subtest, 'focus right', $win0->name; subtest 'focus left', \&focus_subtest, 'focus left', $win2->name; +subtest 'kill doesn\'t produce focus event', \&kill_subtest, '[id=' . $win1->id . '] kill', $win1->name; done_testing; -- 2.39.2