Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(652)

Unified Diff: LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html

Issue 1019273002: Revert "Tweaks layoutTests to use eventSender.setTouchPointRadius correctly" (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
diff --git a/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html b/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
index 2fa8584d5fc2c7b291386debebfc1215bf54f3c8..da82ad3ba89ea3f94d84f4da7847e2b8e9dd8b5c 100644
--- a/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
+++ b/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
@@ -21,9 +21,7 @@
function sendTouchStart(x, y, radiusX, radiusY)
{
eventSender.clearTouchPoints();
- eventSender.addTouchPoint(x, y);
- if (eventSender.setTouchPointRadius)
- eventSender.setTouchPointRadius(0, radiusX, radiusY, false);
+ eventSender.addTouchPoint(x, y, radiusX, radiusY);
eventSender.touchStart();
}
@@ -39,28 +37,22 @@
debug("===Initial Zoom===");
sendTouchStart(100, 100, 4, 10);
- if (eventSender.setTouchPointRadius) {
- shouldBe("radiusX", "4");
- shouldBe("radiusY", "10");
- }
+ shouldBe("radiusX", "4");
+ shouldBe("radiusY", "10");
window.internals.setZoomFactor(2.0);
debug("===Zoom 2X===");
sendTouchStart(100, 100, 4, 10);
- if (eventSender.setTouchPointRadius) {
- shouldBe("radiusX", "2");
- shouldBe("radiusY", "5");
- }
+ shouldBe("radiusX", "2");
+ shouldBe("radiusY", "5");
window.internals.setZoomFactor(0.5);
debug("===Zoom 0.5X===");
sendTouchStart(100, 100, 4, 10);
- if (eventSender.setTouchPointRadius) {
- shouldBe("radiusX", "8");
- shouldBe("radiusY", "20");
- }
+ shouldBe("radiusX", "8");
+ shouldBe("radiusY", "20");
endTest();

Powered by Google App Engine
This is Rietveld 408576698