Merge lp:~mkhu/compiz/compiz-fix_gwd_crash_on_activating_actionmenu into lp:compiz/0.9.8

[Micheal Hsu]

for an application window without decorations, pressing Alt+Space failed to activate the action menu on the top left corner of the window and also causes the gtk-window-decorator to crash.

the crash is caused by a miscalculation in actionmenu.c: position_action_menu(...)
function (*theme_get_button_position) is not suitable for windows without decorations.
so first we need to check if the window is decorated or not.
most (non-overrided) windows use _MOTIF_WM_HINTS to indicate they're not to be decorated.
thus get_mwm_prop(...) in util.c is quite handy.

NOTE: (*theme_get_button_position) is actually get_button_position(...) in cairo.c

BTW, how to propose merge to both lp:compiz/0.9.8 and lp:compiz/0.9.9 in a single pass

[Daniel van Vugt (vanvugt)]

Excellent, thanks.

Please:
1. Log a bug for the crash and link the branch to the bug.
2. Change the indentation to 4 spaces (not tabs except every second level to replace 8-spaces).
3. Propose for merging to lp:compiz only. That's 0.9.9

review: Needs Resubmitting

Unmerged revisions

3412. By hukang <email address hidden> on 2012-11-06

fix gtk-window-decorator crash when pressing Alt+Space on windows without decorations (e.g. google-chrome)

=== modified file 'gtk/window-decorator/actionmenu.c'
--- gtk/window-decorator/actionmenu.c	2011-05-07 08:58:10 +0000
+++ gtk/window-decorator/actionmenu.c	2012-11-06 06:42:21 +0000
@@ -41,9 +41,12 @@
 
     wnck_window_get_client_window_geometry (win, x, y, &width, &height);
 
-    if ((*theme_get_button_position) (d, BUTTON_MENU, width, height,
+    if (get_mwm_prop (wnck_window_get_xid (win)))
+    {
+    	if ((*theme_get_button_position) (d, BUTTON_MENU, width, height,
 				      &bx, &by, &width, &height))
-	*x = *x - frame->win_extents.left + bx;
+		*x = *x - frame->win_extents.left + bx;
+    }
 
     gwd_decor_frame_unref (frame);