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

Unified Diff: LayoutTests/fast/events/touch/multi-touch-inside-iframes.html

Issue 259413003: Correctly handle touch events that contain touches not previously reported to blink (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rjkroege cr Created 6 years, 8 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
« no previous file with comments | « no previous file | LayoutTests/fast/events/touch/multi-touch-inside-iframes-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/events/touch/multi-touch-inside-iframes.html
diff --git a/LayoutTests/fast/events/touch/multi-touch-inside-iframes.html b/LayoutTests/fast/events/touch/multi-touch-inside-iframes.html
index 655402f9f2af080788856e73346e0ad9ae16bfa2..d7c9fe8f1371bd8309d380755b958f159edd2b9b 100644
--- a/LayoutTests/fast/events/touch/multi-touch-inside-iframes.html
+++ b/LayoutTests/fast/events/touch/multi-touch-inside-iframes.html
@@ -56,6 +56,7 @@ function onTouch(event, receiver)
shouldBe("ev.targetTouches.length", "1");
shouldBe("ev.touches[0].pageX", "50");
shouldBe("ev.touches[0].pageY", "50");
+ shouldBe('ev.touches[0].identifier', '0');
shouldBeEqualToString('ev.touches[0].target.ownerDocument.title', 'iframe1');
shouldBeEqualToString('ev.changedTouches[0].target.ownerDocument.title', 'iframe1');
shouldBeEqualToString('ev.targetTouches[0].target.ownerDocument.title', 'iframe1');
@@ -64,21 +65,30 @@ function onTouch(event, receiver)
shouldBeEqualToString('ev.type', 'touchmove');
shouldBeEqualToString('touchreceiver', 'iframe1');
shouldBe("ev.changedTouches.length", "1");
- shouldBe("ev.touches.length", "1");
+ shouldBe("ev.touches.length", "2");
shouldBe("ev.targetTouches.length", "1");
shouldBe("ev.touches[0].pageX", "150");
shouldBe("ev.touches[0].pageY", "150");
+ shouldBe('ev.touches[0].identifier', '0');
+ shouldBe('ev.touches[1].identifier', '1');
+ shouldBe('ev.changedTouches[0].identifier', '0');
shouldBeEqualToString('ev.touches[0].target.ownerDocument.title', 'iframe1');
shouldBeEqualToString('ev.changedTouches[0].target.ownerDocument.title', 'iframe1');
shouldBeEqualToString('ev.targetTouches[0].target.ownerDocument.title', 'iframe1');
+ shouldBeEqualToString('ev.touches[1].target.nodeName', '#document');
+ shouldBeEqualToString('ev.touches[1].target.title', 'iframe1');
break;
case 2:
shouldBeEqualToString('ev.type', 'touchend');
shouldBeEqualToString('touchreceiver', 'iframe1');
shouldBe("ev.changedTouches.length", "1");
- shouldBe("ev.touches.length", "0");
+ shouldBe("ev.touches.length", "1");
shouldBe("ev.targetTouches.length", "0");
shouldBeEqualToString('ev.changedTouches[0].target.ownerDocument.title', 'iframe1');
+ shouldBe('ev.changedTouches[0].identifier', '0');
+ shouldBeEqualToString('ev.touches[0].target.nodeName', '#document');
+ shouldBeEqualToString('ev.touches[0].target.title', 'iframe1');
+ shouldBe('ev.touches[0].identifier', '1');
break;
case 3:
shouldBeEqualToString('ev.type', 'touchstart');
« no previous file with comments | « no previous file | LayoutTests/fast/events/touch/multi-touch-inside-iframes-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698