[Libreoffice] [REVIEW] fdo#32552 (alt key opens "File"-menu)

Hello,

Here is small patch to close the tracker 32552 (alt key opens "File"-menu).
I removed the block quoted in the comments, it compiles but I didn't 
test it. I use Gnome and so Gtk on my Debian.
Could someone confirm it's ok (or not) ?

Julien.
(LGPLv3+ / MPL)

commit 92ec3bb031d94cfc8658ee7e5136e15acafa0987
Author: Julien Nabet <serval2412@yahoo.fr>
Date:   Tue Feb 15 22:05:49 2011 +0100

    fdo 32552

diff --git a/vcl/unx/source/window/salframe.cxx b/vcl/unx/source/window/salframe.cxx
index f52f780..56b4b54 100644
--- a/vcl/unx/source/window/salframe.cxx
+++ b/vcl/unx/source/window/salframe.cxx
@@ -3294,25 +3294,6 @@ long X11SalFrame::HandleKeyEvent( XKeyEvent *pEvent )
 
         int nRet = CallCallback( SALEVENT_KEYMODCHANGE, &aModEvt );
 
-        // emulate KEY_MENU
-        if ( ( (nKeySym == XK_Alt_L) || (nKeySym == XK_Alt_R) ) &&
-             ( (nModCode & ~(KEY_MOD3|KEY_MOD2)) == 0 ) )
-        {
-            if( pEvent->type == XLIB_KeyPress )
-                mbKeyMenu = true;
-            else if( mbKeyMenu )
-            {
-                // simulate KEY_MENU
-                aKeyEvt.mnCode    = KEY_MENU | nModCode;
-                aKeyEvt.mnRepeat   = 0;
-                aKeyEvt.mnTime    = pEvent->time;
-                aKeyEvt.mnCharCode = 0;
-                nRet = CallCallback( SALEVENT_KEYINPUT, &aKeyEvt );
-                nRet = CallCallback( SALEVENT_KEYUP, &aKeyEvt );
-            }
-        }
-        else
-            mbKeyMenu = false;
         return nRet;
     }