Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-10 03:45:46 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1231256 (part 11) - Remove unused parameter from HidePopupsInList(). r=roc.

Browse Source 
--HG--
extra : rebase_source : 805d5cadfdd1cd357a1296d1ec2e6dc192fc4597
This commit is contained in:
Nicholas Nethercote 2015-12-08 20:06:04 -08:00
parent a96e4ed3ee
commit 59dfcb9a66
2 changed files with 7 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
layout/xul/nsXULPopupManager.cpp
View File

@ -1196,8 +1196,7 @@ nsXULPopupManager::HidePopupAfterDelay(nsMenuPopupFrame* aPopup)
}
void
nsXULPopupManager::HidePopupsInList(const nsTArray<nsMenuPopupFrame *> &aFrames,
bool aDeselectMenu)
nsXULPopupManager::HidePopupsInList(const nsTArray<nsMenuPopupFrame *> &aFrames)
{
// Create a weak frame list. This is done in a separate array with the
// right capacity predetermined, otherwise the array would get resized and
@ -1271,7 +1270,7 @@ nsXULPopupManager::HidePopupsInDocShell(nsIDocShellTreeItem* aDocShellToHide)
item = parent;
}
HidePopupsInList(popupsToHide, true);
HidePopupsInList(popupsToHide);
}
void
@ -1315,7 +1314,7 @@ nsXULPopupManager::ExecuteMenu(nsIContent* aMenu, nsXULMenuCommandEvent* aEvent)
// Now hide the popups. If the closemenu mode is auto, deselect the menu,
// otherwise only one popup is closing, so keep the parent menu selected.
HidePopupsInList(popupsToHide, cmm == CloseMenuMode_Auto);
HidePopupsInList(popupsToHide);
}
aEvent->SetCloseMenuMode(cmm);
@ -1792,7 +1791,7 @@ nsXULPopupManager::PopupDestroyed(nsMenuPopupFrame* aPopup)
item = item->GetParent();
}
HidePopupsInList(popupsToHide, false);
HidePopupsInList(popupsToHide);
}
bool

9
layout/xul/nsXULPopupManager.h
View File

@ -636,12 +636,9 @@ protected:
// return the topmost menu, skipping over invisible popups
nsMenuChainItem* GetTopVisibleMenu();
// Hide all of the visible popups from the given list. aDeselectMenu
// indicates whether to deselect the menu of popups when hiding; this
// flag is passed as the first argument to HidePopup. This function
// can cause style changes and frame destruction.
void HidePopupsInList(const nsTArray<nsMenuPopupFrame *> &aFrames,
bool aDeselectMenu);
// Hide all of the visible popups from the given list. This function can
// cause style changes and frame destruction.
void HidePopupsInList(const nsTArray<nsMenuPopupFrame *> &aFrames);
// set the event that was used to trigger the popup, or null to clear the
// event details. aTriggerContent will be set to the target of the event.