Clean up includes

Summary:
* effects.h includes client.h and forward declares Client, both at the
  same time. Thus, delete the include;

* the blur effect includes effects.h. That, most likely, is a leftover
  after 3f5bf65a9e.

Test Plan: Compiles.

Reviewers: #kwin, davidedmundson

Reviewed By: #kwin, davidedmundson

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D15191

autotests/integration/effects/slidingpopups_test.cpp

@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "kwin_wayland_test.h"
+#include "client.h"
 #include "composite.h"
 #include "effects.h"
 #include "effectloader.h"

autotests/integration/effects/translucency_test.cpp

@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "kwin_wayland_test.h"
+#include "client.h"
 #include "composite.h"
 #include "effects.h"
 #include "effectloader.h"

autotests/integration/effects/wobbly_shade_test.cpp

@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "kwin_wayland_test.h"
+#include "client.h"
 #include "composite.h"
 #include "cursor.h"
 #include "effects.h"

autotests/integration/move_resize_window_test.cpp

@@ -22,6 +22,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #include "atoms.h"
 #include "platform.h"
 #include "abstract_client.h"
+#include "client.h"
 #include "cursor.h"
 #include "effects.h"
 #include "screens.h"

autotests/integration/x11_client_test.cpp

@@ -19,6 +19,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "kwin_wayland_test.h"
 #include "atoms.h"
+#include "client.h"
 #include "composite.h"
 #include "effects.h"
 #include "effectloader.h"

effects.h

@@ -24,12 +24,13 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 #include "kwineffects.h"
 
-#include "client.h"
 #include "scene.h"
 
 #include <QHash>
 #include <Plasma/FrameSvg>
 
+#include <memory>
+
 namespace Plasma {
 class Theme;
 }
@@ -53,10 +54,12 @@ class AbstractThumbnailItem;
 class DesktopThumbnailItem;
 class WindowThumbnailItem;
 
+class AbstractClient;
 class Client;
 class Compositor;
 class Deleted;
 class EffectLoader;
+class Toplevel;
 class Unmanaged;
 class WindowPropertyNotifyX11Filter;
 

effects/blur/blur.cpp

@@ -20,11 +20,11 @@
  */
 
 #include "blur.h"
-#include "effects.h"
 #include "blurshader.h"
 // KConfigSkeleton
 #include "blurconfig.h"
 
+#include <QGuiApplication>
 #include <QMatrix4x4>
 #include <QLinkedList>
 #include <QScreen> // for QGuiApplication

pointer_input.cpp

@@ -19,6 +19,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "pointer_input.h"
 #include "platform.h"
+#include "client.h"
 #include "effects.h"
 #include "input_event.h"
 #include "input_event_spy.h"

window_property_notify_x11_filter.cpp

@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "window_property_notify_x11_filter.h"
+#include "client.h"
 #include "effects.h"
 #include "unmanaged.h"
 #include "workspace.h"