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

Unified Diff: LayoutTests/fast/events/scroll-event-handler-reused-window.html

Issue 240343002: Track event handlers on windows (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed assertions not to go through the document. Created 6 years, 7 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/scroll-event-handler-reused-window.html
diff --git a/LayoutTests/fast/events/scroll-event-handler-reused-window.html b/LayoutTests/fast/events/scroll-event-handler-reused-window.html
new file mode 100644
index 0000000000000000000000000000000000000000..97f75078b42606b5e59795e116a70d395f0887a6
--- /dev/null
+++ b/LayoutTests/fast/events/scroll-event-handler-reused-window.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
Rick Byers 2014/05/06 20:54:39 missing expected output for this test
Sami 2014/05/07 10:36:41 Oops, added.
+<script src="../../resources/js-test.js"></script>
+<script>
+description('Test tracking event handlers with a reused DOMWindow.');
Rick Byers 2014/05/06 20:54:39 can you elaborate on what you mean by "reused DOMW
Sami 2014/05/07 10:36:41 Sure, I've added a comment here: // |window.open|
Rick Byers 2014/05/07 13:28:21 Fascinating, thanks!
+
+if (window.testRunner) {
+ testRunner.setCanOpenWindows();
+ testRunner.waitUntilDone();
+}
+
+var newWindow = window.open('resources/conclude-test-in-parent.html');
+newWindow.onscroll = function() {};
+
+function concludeTest() {
+ shouldBe('window.internals.scrollEventHandlerCount(newWindow.document)', '1');
+ testRunner.notifyDone();
+}
+</script>
+</body>

Powered by Google App Engine
This is Rietveld 408576698