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

Unified Diff: tests/html/element_test.dart

Issue 52443007: Allow matchingTarget to also include the currentTarget. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 2 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: tests/html/element_test.dart
diff --git a/tests/html/element_test.dart b/tests/html/element_test.dart
index 43d0bc5f02618410064b6669faa21b0fdc3598b9..3d6c8ee242a4670f53dd2d0187f5596a93255fc2 100644
--- a/tests/html/element_test.dart
+++ b/tests/html/element_test.dart
@@ -780,6 +780,13 @@ main() {
expect(event.target, clickOne);
expect(event.matchingTarget, selectorOne);
}));
+
+ selectorOne.onClick.matches('.selector').listen(expectAsync1(
+ (Event event) {
+ expect(event.currentTarget, selectorOne);
+ expect(event.target, clickOne);
+ expect(event.matchingTarget, selectorOne);
+ }));
clickOne.click();
Element elem = new Element.div()..classes.addAll(['a', 'b']);

Powered by Google App Engine
This is Rietveld 408576698