Fixing more keyboard navigation glitches in XP menus.

layout/xul/base/src/nsMenuFrame.cpp

@@ -243,8 +243,12 @@ nsMenuFrame::HandleEvent(nsIPresContext& aPresContext,
         if (isActive) cancel = PR_FALSE;
       }
       
-      if (cancel)
-        mMenuParent->SetCurrentMenuItem(nsnull);
+      if (cancel) {
+        if (IsMenu() && !isMenuBar && mMenuOpen) {
+          // Submenus don't get closed up.
+        }
+        else mMenuParent->SetCurrentMenuItem(nsnull);
+      }
     }
   }
   else if (aEvent->message == NS_MOUSE_MOVE && mMenuParent) {
@@ -480,6 +484,7 @@ void
 nsMenuFrame::Notify(nsITimer* aTimer)
 {
   // Our timer has fired.
+  if (aTimer == mOpenTimer.get()) {
     if (!mMenuOpen && mMenuParent) {
       nsAutoString active = "";
       mContent->GetAttribute(kNameSpaceID_None, nsXULAtoms::menuactive, active);
@@ -488,6 +493,10 @@ nsMenuFrame::Notify(nsITimer* aTimer)
         OpenMenu(PR_TRUE);
       }
     }
+    mOpenTimer->Cancel();
+    mOpenTimer = nsnull;
+  }
+  
   mOpenTimer = nsnull;
 }
 

layout/xul/base/src/nsMenuPopupFrame.cpp

@@ -486,12 +486,17 @@ nsMenuPopupFrame::KeyboardNavigation(PRUint32 aDirection, PRBool& aHandledFlag)
 
     aHandledFlag = PR_TRUE;
   }
-  else if (aDirection == NS_VK_LEFT && mCurrentMenu &&
-           menuFrame->IsMenu()) {
-    // Close it up.
+  else if (mCurrentMenu && menuFrame->IsMenu() && menuFrame->IsOpen()) {
     aHandledFlag = PR_TRUE;
+    if (aDirection == NS_VK_LEFT) {
+      // Close it up.
       menuFrame->OpenMenu(PR_FALSE);
     }
+    else if (aDirection == NS_VK_RIGHT) {
+      // Select the first item.
+      menuFrame->SelectFirstItem();
+    }
+  }
 }
 
 NS_IMETHODIMP