x11: Consolidate root window event setting in one place

No need to duplicate things in xsettings code.

diff --git a/gdk/x11/gdkscreen-x11.c b/gdk/x11/gdkscreen-x11.c
index 4ab66c3575ceb8286961b893bc48d3197811cc5e..fd370207bc43d1fd06912a5ddb869a24fde426a9 100644 (file)
--- a/gdk/x11/gdkscreen-x11.c
+++ b/gdk/x11/gdkscreen-x11.c
@@ -1095,6 +1095,7 @@ init_randr_support (GdkScreen *screen)
 {
   GdkX11Screen *x11_screen = GDK_X11_SCREEN (screen);
 
+  /* NB: This is also needed for XSettings, so don't remove. */
   XSelectInput (GDK_SCREEN_XDISPLAY (screen),
                 x11_screen->xroot_window,
                 StructureNotifyMask);

diff --git a/gdk/x11/xsettings-client.c b/gdk/x11/xsettings-client.c
index 1cc3729bc0002f091fc52d1dc6b7f1e3bd577ac7..bdcf49dce1d97fc027282800fc3e66b0199714c6 100644 (file)
--- a/gdk/x11/xsettings-client.c
+++ b/gdk/x11/xsettings-client.c
@@ -397,17 +397,6 @@ read_settings (XSettingsClient *client)
     g_hash_table_unref (old_list);
 }
 
-static void
-add_events (Display *display,
-           Window   window,
-           long     mask)
-{
-  XWindowAttributes attr;
-
-  XGetWindowAttributes (display, window, &attr);
-  XSelectInput (display, window, attr.your_event_mask | mask);
-}
-
 static Bool
 gdk_xsettings_watch (Window     window,
                     Bool       is_start,
@@ -563,10 +552,6 @@ xsettings_client_new (GdkScreen *screen)
   client->xsettings_atom = atoms[1];
   client->manager_atom = atoms[2];
 
-  /* Select on StructureNotify so we get MANAGER events
-   */
-  add_events (client->display, gdk_x11_window_get_xid (gdk_screen_get_root_window (screen)), StructureNotifyMask);
-
   gdk_xsettings_watch (gdk_x11_window_get_xid (gdk_screen_get_root_window (screen)), True, client->screen);
 
   check_manager_window (client);