[PATCH 3/9] compositor: remove idle_inhibit optimization

[Pekka Paalanen]

With the idle_inhibit optimization, wlsc_compositor_wake() is a no-op if
idle_inhibit > 0.

When the shell is waking up the compositor from SLEEPING state as an
indirect response to input activity, it does not work. The call path is:
notify_key() / notify_button()
	wlsc_compositor_idle_inhibit()
		wlsc_compositor_activity()
			shell->unlock()
				send prepare_lock_surface event
	idle_inhibit++

and when the desktop-shell client responds to the event:

desktop_shell_set_lock_surface() / desktop_shell_unlock()
	wlsc_compositor_wake()
		no-op, because idle_inhibit > 0

Fix this by removing the idle_inhibit check from wlsc_compositor_wake().
The optimization did not work for pointer motion while no keys pressed,
anyway, so the performance hit is probably unobservable.

Now the compositor wakes up also on key or button press.

Signed-off-by: Pekka Paalanen <ppaalanen at gmail.com>
---
 compositor/compositor.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/compositor/compositor.c b/compositor/compositor.c
index 144e329..77b9a99 100644
--- a/compositor/compositor.c
+++ b/compositor/compositor.c
@@ -1323,9 +1323,6 @@ static const struct wl_grab_interface implicit_grab_interface = {
 WL_EXPORT void
 wlsc_compositor_wake(struct wlsc_compositor *compositor)
 {
-	if (compositor->idle_inhibit)
-		return;
-
 	wlsc_compositor_fade(compositor, 0.0);
 	compositor->state = WLSC_COMPOSITOR_ACTIVE;
 
-- 
1.7.3.4