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

Fix regression from bug 244761 so that only user actions will enable onChange after blur b=251833 r/sr=roc

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@159723 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
neil%parkwaycc.co.uk
2004-07-23 08:25:44 +00:00
parent 697a2a863c
commit fc3edfffcf
8 changed files with 50 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
mozilla/layout/forms/nsListControlFrame.cpp
View File

@@ -1488,6 +1488,12 @@ nsListControlFrame::ExtendedSelection(PRInt32 aStartIndex,
PRBool
nsListControlFrame::SingleSelection(PRInt32 aClickedIndex, PRBool aDoToggle)
{
if (mComboboxFrame) {
PRInt32 selectedIndex;
GetSelectedIndex(&selectedIndex);
mComboboxFrame->UpdateRecentIndex(selectedIndex);
}
PRBool wasChanged = PR_FALSE;
// Get Current selection
if (aDoToggle) {
@@ -2425,7 +2431,15 @@ nsListControlFrame::FireOnChange()
nsresult rv = NS_OK;
if (mComboboxFrame) {
if (!mComboboxFrame->NeededToFireOnChange())
// Return hit without changing anything
PRInt32 index = mComboboxFrame->UpdateRecentIndex(-1);
if (index == -1)
return NS_OK;
// See if the selection actually changed
PRInt32 selectedIndex;
GetSelectedIndex(&selectedIndex);
if (index == selectedIndex)
return NS_OK;
}