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

Bug 207637. Don't fire onchange for MouseUp events where we didn't see a mousedown. r+sr=dbaron,a=asa

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@145922 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
roc+%cs.cmu.edu
2003-08-12 16:00:20 +00:00
parent a84edff323
commit 5dc0998c4e
2 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -2794,6 +2794,9 @@ nsListControlFrame::MouseUp(nsIDOMEvent* aMouseEvent)
CaptureMouseEvents(mPresContext, PR_FALSE);
// Notify
if (mChangesSinceDragStart) {
// reset this so that future MouseUps without a prior MouseDown
// won't fire onchange
mChangesSinceDragStart = PR_FALSE;
FireOnChange();
}
#if 0 // XXX - this is a partial fix for Bug 29990

3
mozilla/layout/html/forms/src/nsListControlFrame.cpp
View File

@@ -2794,6 +2794,9 @@ nsListControlFrame::MouseUp(nsIDOMEvent* aMouseEvent)
CaptureMouseEvents(mPresContext, PR_FALSE);
// Notify
if (mChangesSinceDragStart) {
// reset this so that future MouseUps without a prior MouseDown
// won't fire onchange
mChangesSinceDragStart = PR_FALSE;
FireOnChange();
}
#if 0 // XXX - this is a partial fix for Bug 29990