From 85ef2831dbc1b62543ee25f1691b5cfb3d2e6bb1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Fl=C3=B6ser?= <mgraesslin@kde.org>
Date: Sat, 13 Jan 2018 19:55:20 +0100
Subject: [PATCH] Do not call into NETWinInfo in AbstractClient::setSkipPager

Summary:
There's a call to a virtual doSetSkipPager for performing this call and
Client implements it with the code which now is removed.

This fixes a crash when using setSkipPager for a Wayland window.

Test Plan:
Crash which happens with a new test case (see dependent phab
request) is fixed.

Reviewers: #kwin, #plasma

Subscribers: plasma-devel, kwin

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9865
---
 abstract_client.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/abstract_client.cpp b/abstract_client.cpp
index 711190e798..10bd76b664 100644
--- a/abstract_client.cpp
+++ b/abstract_client.cpp
@@ -153,7 +153,6 @@ void AbstractClient::setSkipPager(bool b)
         return;
     m_skipPager = b;
     doSetSkipPager();
-    info->setState(b ? NET::SkipPager : NET::States(0), NET::SkipPager);
     updateWindowRules(Rules::SkipPager);
     emit skipPagerChanged();
 }