Bug 20134 - REGRESSION (2.0.4-3.0.4): No default value set for <input type=range> with an even difference of (max - min)
Summary: REGRESSION (2.0.4-3.0.4): No default value set for <input type=range> with an...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 525.x (Safari 3.1)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Nobody
URL:
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2008-07-22 14:21 PDT by David Kilzer (:ddkilzer)
Modified: 2008-10-03 12:23 PDT (History)
0 users

See Also:


Attachments
Test case (635 bytes, text/html)
2008-07-22 14:21 PDT, David Kilzer (:ddkilzer)
no flags Details
First attempt (3.50 KB, patch)
2008-10-03 04:32 PDT, Rob Buis
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2008-07-22 14:21:05 PDT
* SUMMARY
Setting the min and max values on an <input type=range> element such that the difference between them is even, no default value is set for the input element.

* STEPS TO REPRODUCE
1. Launch Safari/WebKit.
2. Open attached test case.

* EXPECTED RESULTS
An alert dialog with "PASS: '50'" should be displayed and "50" should be displayed in the read-only text field.

* ACTUAL RESULTS
An alert dialog with "FAIL: ''" is displayed and no value is displayed in the read-only text field.

* REGRESSION
Unknown.  Only tested WebKit nightly build r35249.
Comment 1 David Kilzer (:ddkilzer) 2008-07-22 14:21:26 PDT
Created attachment 22433 [details]
Test case
Comment 2 Mark Rowe (bdash) 2008-07-22 17:30:34 PDT
<rdar://problem/6095022>
Comment 3 David Kilzer (:ddkilzer) 2008-07-23 06:58:01 PDT
This is a regression from Safari 2.0.4 (which behaved correctly).  Unfortunately, the earliest nightly build (r11976) also shows the bug, so a bisect of nightly builds can't be done.
Comment 4 Rob Buis 2008-10-03 04:32:18 PDT
Created attachment 24047 [details]
First attempt

I think this patch should fix the problem.
Cheers,

Rob.
Comment 5 Darin Adler 2008-10-03 08:54:33 PDT
Comment on attachment 24047 [details]
First attempt

r=me
Comment 6 Rob Buis 2008-10-03 12:23:23 PDT
Landed in r37247.