Title: [103633] trunk/LayoutTests
- Revision
- 103633
- Author
- commit-qu...@webkit.org
- Date
- 2011-12-23 09:34:34 -0800 (Fri, 23 Dec 2011)
Log Message
[Mac] fast/events/autoscroll-should-not-stop-on-keypress.html fails
https://bugs.webkit.org/show_bug.cgi?id=74538
Patch by Rakesh KN <rakesh...@motorola.com> on 2011-12-23
Reviewed by Alexey Proskuryakov.
* fast/events/autoscroll-should-not-stop-on-keypress-expected.txt:
* fast/events/autoscroll-should-not-stop-on-keypress.html:
Modified test to use iframe for testing autoscroll.
* platform/mac/Skipped:
Removed this test entry.
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (103632 => 103633)
--- trunk/LayoutTests/ChangeLog 2011-12-23 17:18:00 UTC (rev 103632)
+++ trunk/LayoutTests/ChangeLog 2011-12-23 17:34:34 UTC (rev 103633)
@@ -1,3 +1,16 @@
+2011-12-23 Rakesh KN <rakesh...@motorola.com>
+
+ [Mac] fast/events/autoscroll-should-not-stop-on-keypress.html fails
+ https://bugs.webkit.org/show_bug.cgi?id=74538
+
+ Reviewed by Alexey Proskuryakov.
+
+ * fast/events/autoscroll-should-not-stop-on-keypress-expected.txt:
+ * fast/events/autoscroll-should-not-stop-on-keypress.html:
+ Modified test to use iframe for testing autoscroll.
+ * platform/mac/Skipped:
+ Removed this test entry.
+
2011-12-23 Simon Fraser <simon.fra...@apple.com>
Update the test result (whitespace difference).
Modified: trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress-expected.txt (103632 => 103633)
--- trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress-expected.txt 2011-12-23 17:18:00 UTC (rev 103632)
+++ trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress-expected.txt 2011-12-23 17:34:34 UTC (rev 103633)
@@ -1,166 +1,5 @@
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-Lots of text
-PASS document.body.scrollTop + window.innerHeight is document.body.scrollHeight
+
+PASS frame.contentDocument.body.scrollTop + frame.clientHeight is frame.contentDocument.body.scrollHeight
PASS successfullyParsed is true
TEST COMPLETE
Modified: trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress.html (103632 => 103633)
--- trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress.html 2011-12-23 17:18:00 UTC (rev 103632)
+++ trunk/LayoutTests/fast/events/autoscroll-should-not-stop-on-keypress.html 2011-12-23 17:34:34 UTC (rev 103633)
@@ -6,14 +6,76 @@
<script>
window.jsTestIsAsync = true;
+var frame;
function test() {
+ frame = document.getElementById('frame');
+ var doc = frame.contentDocument.open();
+ var htmlString = "Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> \
+ Lots of text</br>Lots of text</br> ";
+
+ doc.write(htmlString);
+ doc.close();
+ frame.contentDocument.body.style.height = "2000px";
+
if (window.eventSender) {
- var height = document.body.scrollHeight + 100;
+ var xPoint = frame.scrollLeft + 100;
+ var yPoint = frame.scrollTop + 100;
eventSender.dragMode = false;
- eventSender.mouseMoveTo(20, 100);
+ eventSender.mouseMoveTo(xPoint, yPoint);
eventSender.mouseDown();
- eventSender.mouseMoveTo(20, height);
- setTimeout(testAutoScroll, 200);
+ eventSender.mouseMoveTo(xPoint, frame.offsetHeight + 100);
+ setTimeout(testAutoScroll, 1500);
eventSender.keyDown('a');
} else {
debug("Test manually that scrolling does not stop if we press a key while autoscroll is happening.");
@@ -21,95 +83,15 @@
}
function testAutoScroll() {
- shouldBe('document.body.scrollTop + window.innerHeight', 'document.body.scrollHeight');
+ shouldBe('frame.contentDocument.body.scrollTop + frame.clientHeight', 'frame.contentDocument.body.scrollHeight');
finishJSTest();
}
successfullyParsed = true;
-</script>
+</script>
</head>
<body _onload_="test()">
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
-Lots of text</br>Lots of text</br>
+<iframe id="frame" style="height: 400px;" frameborder="0"></iframe>
<div id="console"></div>
<script src=""
</body>
Modified: trunk/LayoutTests/platform/mac/Skipped (103632 => 103633)
--- trunk/LayoutTests/platform/mac/Skipped 2011-12-23 17:18:00 UTC (rev 103632)
+++ trunk/LayoutTests/platform/mac/Skipped 2011-12-23 17:34:34 UTC (rev 103633)
@@ -520,9 +520,6 @@
# Tiemout. Needs ChromeClient::hasOpenedPopup() impelementation?
fast/forms/select/menulist-popup-crash.html
-# https://bugs.webkit.org/show_bug.cgi?id=74538
-fast/events/autoscroll-should-not-stop-on-keypress.html
-
# Frame::findString does nothing on pages that prevent selection
# https://bugs.webkit.org/show_bug.cgi?id=40361
editing/text-iterator/findString-selection-disabled.html
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
