diff options
author | Hans de Goede <hdegoede@redhat.com> | 2010-10-17 10:43:28 +0200 |
---|---|---|
committer | Hans de Goede <hdegoede@redhat.com> | 2010-10-18 10:40:51 +0200 |
commit | 0b07ca2775888a10d64e51fb95425ff932fc4422 (patch) | |
tree | 420007c2ca35f9b5c545bdcab193638853c580b2 /client/x11/red_window.cpp | |
parent | 203a39761f7da9547e5856efbb2b5f51ac63f369 (diff) | |
download | spice-0b07ca2775888a10d64e51fb95425ff932fc4422.tar.gz spice-0b07ca2775888a10d64e51fb95425ff932fc4422.tar.xz spice-0b07ca2775888a10d64e51fb95425ff932fc4422.zip |
spicec-x11: Change WmSizeHints in fullscreen mode
Some window managers will ignore the fullscreen hint, unless WmSizeHints
allow them to resize the window so that they can give it the size of
the roo-window. This fixes fullscreen mode in compiz.
Diffstat (limited to 'client/x11/red_window.cpp')
-rw-r--r-- | client/x11/red_window.cpp | 50 |
1 files changed, 32 insertions, 18 deletions
diff --git a/client/x11/red_window.cpp b/client/x11/red_window.cpp index e7b68151..8f6535b9 100644 --- a/client/x11/red_window.cpp +++ b/client/x11/red_window.cpp @@ -1063,18 +1063,27 @@ void RedWindow_p::set_glx(int width, int height) } #endif // USE_OGL -void RedWindow_p::set_minmax(PixelsSource_p& pix_source, int width, int height) +void RedWindow_p::set_minmax(PixelsSource_p& pix_source) { //todo: auto res XSizeHints* size_hints = XAllocSizeHints(); ASSERT(size_hints); size_hints->flags = PMinSize | PMaxSize; - size_hints->min_width = size_hints->max_width = width; - size_hints->min_height = size_hints->max_height = height; + if (_red_window->_type == RedWindow::TYPE_FULLSCREEN) { + /* Some window managers won't allow full screen mode with a fixed + width / height */ + size_hints->min_width = 1; + size_hints->max_width = 65535; + size_hints->min_height = 1; + size_hints->max_height = 65535; + } else { + size_hints->min_width = size_hints->max_width = _width; + size_hints->min_height = size_hints->max_height = _height; + } XSetWMNormalHints(x_display, _win, size_hints); XFree(size_hints); - pix_source.x_drawable.height = height; - pix_source.x_drawable.width = width; + pix_source.x_drawable.height = _height; + pix_source.x_drawable.width = _width; } Cursor RedWindow_p::create_invisible_cursor(Window window) @@ -1100,6 +1109,8 @@ RedWindow_p::RedWindow_p() , _ignore_foucs (false) , _pointer_in_window (false) , _ignore_pointer (false) + ,_width (200) + ,_height (200) { } @@ -1145,8 +1156,8 @@ void RedWindow_p::destroy(RedWindow& red_window, PixelsSource_p& pix_source) } } -void RedWindow_p::create(RedWindow& red_window, PixelsSource_p& pix_source, int x, int y, - unsigned int width, unsigned int height, int in_screen) +void RedWindow_p::create(RedWindow& red_window, PixelsSource_p& pix_source, + int x, int y, int in_screen) { Window window = None; Cursor cursor = None; @@ -1168,7 +1179,7 @@ void RedWindow_p::create(RedWindow& red_window, PixelsSource_p& pix_source, int win_attributes.colormap = _colormap; mask |= CWColormap; window = XCreateWindow(x_display, root_window, x, y, - width, height, 0, XPlatform::get_vinfo()[in_screen]->depth, + _width, _height, 0, XPlatform::get_vinfo()[in_screen]->depth, InputOutput, XPlatform::get_vinfo()[in_screen]->visual, mask, &win_attributes); XUnlockDisplay(x_display); @@ -1223,11 +1234,12 @@ void RedWindow_p::create(RedWindow& red_window, PixelsSource_p& pix_source, int _show_pos.y = y; _visibale = false; _expect_parent = false; + _red_window = &red_window; pix_source.type = PIXELS_SOURCE_TYPE_X_DRAWABLE; pix_source.x_drawable.drawable = window; pix_source.x_drawable.screen = _screen; pix_source.x_drawable.gc = gc; - set_minmax(pix_source, width, height); + set_minmax(pix_source); sync(); } @@ -1236,17 +1248,12 @@ void RedWindow_p::migrate(RedWindow& red_window, PixelsSource_p& pix_source, int if (to_screen == _screen) { return; } - XWindowAttributes attrib; - if (!XGetWindowAttributes(x_display, _win, &attrib)) { - THROW("get attributes failed"); - } XTextProperty text_pro; XLockDisplay(x_display); bool valid_title = XGetWMName(x_display, _win, &text_pro) && text_pro.value; XUnlockDisplay(x_display); destroy(red_window, pix_source); - create(red_window, pix_source, _show_pos.x, _show_pos.y, attrib.width, attrib.height, - to_screen); + create(red_window, pix_source, _show_pos.x, _show_pos.y, to_screen); if (valid_title) { XSetWMName(x_display, _win, &text_pro); XFree(text_pro.value); //??? @@ -1304,7 +1311,7 @@ RedWindow::RedWindow(RedWindow::Listener& listener, int screen) , _menu (NULL) { ASSERT(x_display); - create(*this, *(PixelsSource_p*)get_opaque(), 0, 0, 200, 200, + create(*this, *(PixelsSource_p*)get_opaque(), 0, 0, (screen == DEFAULT_SCREEN) ? DefaultScreen(x_display) : screen); } @@ -1505,6 +1512,9 @@ void RedWindow::show(int screen_id) _listener.post_migrate(); } + /* We must update min/max for fullscreen / normal switching */ + set_minmax(*(PixelsSource_p*)get_opaque()); + if (_type == TYPE_FULLSCREEN) { Atom state[2]; state[0] = wm_state_above; @@ -1621,7 +1631,9 @@ static void send_expose(Window window, int width, int height) void RedWindow::move_and_resize(int x, int y, int width, int height) { - set_minmax(*(PixelsSource_p*)get_opaque(), width, height); + _width = width; + _height = height; + set_minmax(*(PixelsSource_p*)get_opaque()); XMoveResizeWindow(x_display, _win, x, y, width, height); _show_pos.x = x; _show_pos.y = y; @@ -1639,7 +1651,9 @@ void RedWindow::move(int x, int y) void RedWindow::resize(int width, int height) { - set_minmax(*(PixelsSource_p*)get_opaque(), width, height); + _width = width; + _height = height; + set_minmax(*(PixelsSource_p*)get_opaque()); XResizeWindow(x_display, _win, width, height); if (_visibale) { send_expose(_win, width, height); |