Bug 6942

Summary: Assertion failure in HTMLSelectElementImpl::selectedIndex (m_multiple)
Product: WebKit Reporter: mitz
Component: FormsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: adele, sullivan
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://bugzilla.opendarwin.org/enter_bug.cgi?product=WebKit
Attachments:
Description Flags
Remove the assert adele: review+

mitz
Reported 2006-01-30 14:13:02 PST
The assert at the end of HTMLSelectElementImpl::selectedIndex fails when no item is selected. It went unnoticed before because it was a Q_ASSERT, but in r12471 it was changed to an ASSERT. Anyway, it is bogus since -1 is the correct value to return when no option is selected.
Attachments
Remove the assert (3.09 KB, patch)
2006-01-30 14:36 PST, mitz 		adele: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-01-30 14:36:03 PST
Created attachment 6119 [details] Remove the assert Not sure the test is necessary
Adele Peterson
Comment 2 2006-01-30 14:36:54 PST
Comment on attachment 6119 [details] Remove the assert r=me
Darin Adler
Comment 3 2006-01-30 23:07:04 PST
Oops, yes, I was wondering which of the Q_ASSERT that I changed to ASSERT were bogus. Sorry, my mistake.
John Sullivan
Comment 4 2006-01-31 11:00:33 PST
*** Bug 6974 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.