From a0f5caf3e86fc9ef80c977ef64303235f7edcee9 Mon Sep 17 00:00:00 2001
From: "hyatt%netscape.com"
 <hyatt%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 12 Aug 1999 21:20:04 +0000
Subject: [PATCH] A minor addendum to the bug fix I checked in a few minutes
 ago.

git-svn-id: svn://10.0.0.236/trunk@43254 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/layout/xul/base/src/nsMenuFrame.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/mozilla/layout/xul/base/src/nsMenuFrame.cpp b/mozilla/layout/xul/base/src/nsMenuFrame.cpp
index c104af3721b..8dbf4c63ac5 100644
--- a/mozilla/layout/xul/base/src/nsMenuFrame.cpp
+++ b/mozilla/layout/xul/base/src/nsMenuFrame.cpp
@@ -884,24 +884,24 @@ nsMenuFrame::BuildAcceleratorText(nsString& aAccelString)
 	  
   PRBool prependPlus = PR_FALSE;
 
-  if(commandValue != "") {
+  if(commandValue != "" && commandValue != "false") {
     prependPlus = PR_TRUE;
 	  aAccelString += "Ctrl"; // Hmmm. Kinda defeats the point of having an abstraction.
   }
 
-  if(controlValue != "") {
+  if(controlValue != "" && controlValue != "false") {
     prependPlus = PR_TRUE;
 	  aAccelString += "Ctrl";
   }
 
-  if(shiftValue != "") {
+  if(shiftValue != "" && shiftValue != "false") {
     if (prependPlus)
       aAccelString += "+";
     prependPlus = PR_TRUE;
     aAccelString += "Shift";
   }
 
-  if (altValue != "") {
+  if (altValue != "" && altValue != "false") {
 	  if (prependPlus)
       aAccelString += "+";
     prependPlus = PR_TRUE;