- // XXX: manage the window, i.e. grab events n shit
- Openbox::instance->addClient(e.window, new OBClient(e.window));
+ int screen = INT_MAX;
+
+ for (int i = 0; i < ScreenCount(otk::OBDisplay::display); ++i)
+ if (otk::OBDisplay::screenInfo(i)->getRootWindow() == e.parent) {
+ screen = i;
+ break;
+ }
+
+ if (screen >= ScreenCount(otk::OBDisplay::display)) {
+ /*
+ we got a map request for a window who's parent isn't root. this
+ can happen in only one circumstance:
+
+ a client window unmapped a managed window, and then remapped it
+ somewhere between unmapping the client window and reparenting it
+ to root.
+
+ regardless of how it happens, we need to find the screen that
+ the window is on
+ */
+ XWindowAttributes wattrib;
+ if (! XGetWindowAttributes(otk::OBDisplay::display, e.window,
+ &wattrib)) {
+ // failed to get the window attributes, perhaps the window has
+ // now been destroyed?
+ return;
+ }
+
+ for (int i = 0; i < ScreenCount(otk::OBDisplay::display); ++i)
+ if (otk::OBDisplay::screenInfo(i)->getRootWindow() == wattrib.root) {
+ screen = i;
+ break;
+ }
+ }
+
+ assert(screen < ScreenCount(otk::OBDisplay::display));
+
+ Openbox::instance->screen(screen)->manageWindow(e.window);