Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 291099 menuitem context menus arrow keys doubled r=aaronlev sr=dbaron a=asa

Browse Source 
git-svn-id: svn://10.0.0.236/branches/MOZILLA_1_8_BRANCH@182245 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
neil%parkwaycc.co.uk
2005-10-14 09:37:22 +00:00
parent f5cb3dc429
commit 2cda4f73c0
1 changed files with 16 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
mozilla/layout/xul/base/src/nsPopupSetFrame.cpp
View File

@@ -494,21 +494,23 @@ nsPopupSetFrame::OpenPopup(nsPopupFrameList* aEntry, PRBool aActivateFlag)
ActivatePopup(aEntry, PR_TRUE);
// register the rollup listeners, etc, but not if we're a tooltip
nsIFrame* activeChild = aEntry->mPopupFrame;
nsIMenuParent* childPopup = nsnull;
if (activeChild)
CallQueryInterface(activeChild, &childPopup);
if (!aEntry->mPopupType.EqualsLiteral("tooltip"))
if (!aEntry->mPopupType.EqualsLiteral("tooltip")) {
nsIFrame* activeChild = aEntry->mPopupFrame;
nsIMenuParent* childPopup = nsnull;
if (activeChild)
CallQueryInterface(activeChild, &childPopup);
// Tooltips don't get keyboard navigation
if (childPopup && !nsMenuFrame::sDismissalListener) {
// First check and make sure this popup wants keyboard navigation
nsAutoString property;
aEntry->mPopupContent->GetAttr(kNameSpaceID_None, nsXULAtoms::ignorekeys, property);
if (!property.EqualsLiteral("true"))
childPopup->InstallKeyboardNavigator();
}
UpdateDismissalListener(childPopup);
// First check and make sure this popup wants keyboard navigation
nsAutoString property;
// Tooltips don't get keyboard navigation
aEntry->mPopupContent->GetAttr(kNameSpaceID_None, nsXULAtoms::ignorekeys, property);
if (!property.EqualsLiteral("true") &&
childPopup &&
!aEntry->mPopupType.EqualsLiteral("tooltip"))
childPopup->InstallKeyboardNavigator();
}
}
else {
if (aEntry->mCreateHandlerSucceeded && !OnDestroy(aEntry->mPopupContent))