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

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

Issue 990193002: Tweaks layoutTests to use eventSender.setTouchPointRadius correctly (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: LayoutTests fixed (expected results) 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 da82ad3ba89ea3f94d84f4da7847e2b8e9dd8b5c..2fa8584d5fc2c7b291386debebfc1215bf54f3c8 100644
--- a/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
+++ b/LayoutTests/fast/events/touch/touch-browser-zoom-scales-radius.html
@@ -21,7 +21,9 @@
function sendTouchStart(x, y, radiusX, radiusY)
{
eventSender.clearTouchPoints();
- eventSender.addTouchPoint(x, y, radiusX, radiusY);
+ eventSender.addTouchPoint(x, y);
+ if (eventSender.setTouchPointRadius)
+ eventSender.setTouchPointRadius(0, radiusX, radiusY, false);
eventSender.touchStart();
}
@@ -37,22 +39,28 @@
debug("===Initial Zoom===");
sendTouchStart(100, 100, 4, 10);
- shouldBe("radiusX", "4");
- shouldBe("radiusY", "10");
+ if (eventSender.setTouchPointRadius) {
+ shouldBe("radiusX", "4");
+ shouldBe("radiusY", "10");
+ }
window.internals.setZoomFactor(2.0);
debug("===Zoom 2X===");
sendTouchStart(100, 100, 4, 10);
- shouldBe("radiusX", "2");
- shouldBe("radiusY", "5");
+ if (eventSender.setTouchPointRadius) {
+ shouldBe("radiusX", "2");
+ shouldBe("radiusY", "5");
+ }
window.internals.setZoomFactor(0.5);
debug("===Zoom 0.5X===");
sendTouchStart(100, 100, 4, 10);
- shouldBe("radiusX", "8");
- shouldBe("radiusY", "20");
+ if (eventSender.setTouchPointRadius) {
+ shouldBe("radiusX", "8");
+ shouldBe("radiusY", "20");
+ }
endTest();

Powered by Google App Engine
This is Rietveld 408576698