[maemo-commits] [maemo-commits] r18251 - in projects/haf/trunk/libmatchbox2: . matchbox/core

[From: subversion at stage.maemo.org]

Author: aendrodi
Date: 2009-05-06 09:46:51 +0300 (Wed, 06 May 2009)
New Revision: 18251

Modified:
   projects/haf/trunk/libmatchbox2/ChangeLog
   projects/haf/trunk/libmatchbox2/matchbox/core/mb-wm-client-base.c
Log:
	NB#114681 Application shortcuts in the home view are getting opened
		  when tapped here and there in camera application UI.

	* matchbox/core/mb-wm-client-base.c (mb_wm_client_base_realize):
	  Reverting madam's change not to freak out when we start handling
	  preexisting fullscreen clients.  Breaks initially fullscreen
	  clients when they wants to unmap--we get skip_unmaps wrong.


Modified: projects/haf/trunk/libmatchbox2/ChangeLog
===================================================================
--- projects/haf/trunk/libmatchbox2/ChangeLog	2009-05-05 18:59:52 UTC (rev 18250)
+++ projects/haf/trunk/libmatchbox2/ChangeLog	2009-05-06 06:46:51 UTC (rev 18251)
@@ -1,3 +1,13 @@
+2009-05-05  Adam Endrodi  <adam.endrodi at blumsoft.eu>
+
+	NB#114681 Application shortcuts in the home view are getting opened
+		  when tapped here and there in camera application UI.
+
+	* matchbox/core/mb-wm-client-base.c (mb_wm_client_base_realize):
+	  Reverting madam's change not to freak out when we start handling
+	  preexisting fullscreen clients.  Breaks initially fullscreen
+	  clients when they wants to unmap--we get skip_unmaps wrong.
+
 2009-05-05  Kimmo Hämäläinen  <kimmo.hamalainen at nokia.com>
 
 	Applied a patch from Laszlo for NB#113598:

Modified: projects/haf/trunk/libmatchbox2/matchbox/core/mb-wm-client-base.c
===================================================================
--- projects/haf/trunk/libmatchbox2/matchbox/core/mb-wm-client-base.c	2009-05-05 18:59:52 UTC (rev 18250)
+++ projects/haf/trunk/libmatchbox2/matchbox/core/mb-wm-client-base.c	2009-05-06 06:46:51 UTC (rev 18251)
@@ -166,11 +166,14 @@
   /*
    * We only create a frame window if the client is decorated (decors are
    * constructed in the _init functions, so we can easily test if the frame
-   * is needed or not).  An exception is fullscreen windows, which should not
-   * have a frame, otherwise XRestackWindows() will be irritated.
+   * is needed or not).
+   *
+   * FIXME "An exception is fullscreen windows, which should not have a frame,
+   *        otherwise XRestackWindows() will be irritated."  Preexisting fs
+   *        clients still don't work properly, and it breaks initially fs
+   *        clients.
    */
-  if (client->decor
-      && !(client->window->ewmh_state & MBWMClientWindowEWMHStateFullscreen))
+  if (client->decor)
     {
       if (client->xwin_frame == None)
 	{