Title: [126252] trunk/LayoutTests
Revision
126252
Author
[email protected]
Date
2012-08-21 19:17:33 -0700 (Tue, 21 Aug 2012)

Log Message

range-hit-test-with-padding.html fails unless subpixel layout is on
https://bugs.webkit.org/show_bug.cgi?id=94585

Reviewed by Kent Tamura.

Ports without subpixel layout didn't exactly match the result, because slider thumb width is an odd number and we are dividing it by two.

* fast/forms/range/range-hit-test-with-padding-expected.txt:
* fast/forms/range/range-hit-test-with-padding.html: Adjust the max so it snaps to the step in the middle.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (126251 => 126252)


--- trunk/LayoutTests/ChangeLog	2012-08-22 02:16:02 UTC (rev 126251)
+++ trunk/LayoutTests/ChangeLog	2012-08-22 02:17:33 UTC (rev 126252)
@@ -1,3 +1,15 @@
+2012-08-21  Keishi Hattori  <[email protected]>
+
+        range-hit-test-with-padding.html fails unless subpixel layout is on
+        https://bugs.webkit.org/show_bug.cgi?id=94585
+
+        Reviewed by Kent Tamura.
+
+        Ports without subpixel layout didn't exactly match the result, because slider thumb width is an odd number and we are dividing it by two.
+
+        * fast/forms/range/range-hit-test-with-padding-expected.txt:
+        * fast/forms/range/range-hit-test-with-padding.html: Adjust the max so it snaps to the step in the middle.
+
 2012-08-21  Julien Chaffraix  <[email protected]>
 
         Crash in RenderTableSection::setCellLogicalWidths

Modified: trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding-expected.txt (126251 => 126252)


--- trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding-expected.txt	2012-08-22 02:16:02 UTC (rev 126251)
+++ trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding-expected.txt	2012-08-22 02:17:33 UTC (rev 126252)
@@ -1,6 +1,6 @@
 Clicking middle of input type=range with padding should set the value to middle.
 
-PASS input.value is "500"
+PASS input.value is "25"
 PASS successfullyParsed is true
 
 TEST COMPLETE

Modified: trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding.html (126251 => 126252)


--- trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding.html	2012-08-22 02:16:02 UTC (rev 126251)
+++ trunk/LayoutTests/fast/forms/range/range-hit-test-with-padding.html	2012-08-22 02:17:33 UTC (rev 126252)
@@ -6,7 +6,7 @@
 <body>
 <p id="description">Clicking middle of input type=range with padding should set the value to middle.</p>
 <div id="console"></div>
-<input id="input" type=range min=0 max=1000 step=1 value="0" style="width: 100px; padding: 0 20px;">
+<input id="input" type=range min=0 max=50 step=1 value="0" style="width: 100px; padding: 0 40px;">
 <script>
 var input = document.getElementById("input");
 function clickSlider(offsetLeft) {
@@ -16,8 +16,8 @@
     eventSender.mouseUp();
 }
 
-clickSlider(70); // left padding (20px) + middle (50px)
-shouldBe('input.value', '"500"');
+clickSlider(90); // left padding (40px) + middle (50px)
+shouldBe('input.value', '"25"');
 
 </script>
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to