Fixed a problem with not updating the system menu when it's opened by right-clicking the title bar. Otherwise, the "Show toolbar" option would not always show correct toolbar state.

git-svn-id: svn://svn.code.sf.net/p/tigervnc/code/branches/merge-with-vnc-4.1.1@557 3789f03b-4d11-0410-bbf8-ca57d06f2519
diff --git a/vncviewer/DesktopWindow.cxx b/vncviewer/DesktopWindow.cxx
index 47ddf7d..eee55ff 100644
--- a/vncviewer/DesktopWindow.cxx
+++ b/vncviewer/DesktopWindow.cxx
@@ -490,6 +490,12 @@
     kbd.releaseAllKeys(callback);
     break;
 
+    // -=- If the menu is about to be shown, make sure it's up to date
+
+  case WM_INITMENU:
+    callback->refreshMenu(true);
+    break;
+
     // -=- Handle the extra window menu items
 
     // Pass system menu messages to the callback and only attempt
@@ -524,13 +530,7 @@
       setFullscreen(false);
       break;
 
-      // If the menu is about to be shown, make sure it's up to date
-    case SC_KEYMENU:
-    case SC_MOUSEMENU:
-      callback->refreshMenu(true);
-      break;
-
-    };
+    }
     break;
 
     // Treat all menu commands as system menu commands
@@ -553,6 +553,11 @@
         // If MenuKey is being released then pop-up the menu
         if ((msg == WM_KEYDOWN)) {
           // Make sure it's up to date
+          //
+          // NOTE: Here we call refreshMenu only to grey out Move and Size
+          //       menu items. Other things will be refreshed once again
+          //       while processing the WM_INITMENU message.
+          //
           callback->refreshMenu(false);
 
           // Show it under the pointer