Chromium Code Reviews Chromium Code Reviews
Issue 17063016:
Remove leak of objects between isolated worlds on custom events. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/events/event-isolated-world.html |
| diff --git a/LayoutTests/fast/events/event-isolated-world.html b/LayoutTests/fast/events/event-isolated-world.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..6f79542639c0bbb9312d681fb07a279db72528fb |
| --- /dev/null |
| +++ b/LayoutTests/fast/events/event-isolated-world.html |
| @@ -0,0 +1,71 @@ |
| +<!DOCTYPE html> |
| +<html> |
| +<head> |
| +<script src="../js/resources/js-test-pre.js"></script> |
| +</head> |
| +<body> |
| +<script> |
| +window.jsTestIsAsync = true; |
| +description("Tests that properties of various events do not leak between isolated worlds (bug 85158)."); |
| + |
| +function addListener(eventType, prop) { |
| + document.addEventListener(eventType, function(event) { |
| + documentObject = event[prop]; |
| + |
| + // Object passed into isolated world should be undefined. |
| + shouldBe("documentObject", "undefined"); |
|
haraken
2013/06/27 04:50:27
shouldBeUndefined
jww
2013/06/27 17:44:42
Done.
|
| + |
| + // The property defined in the isolated world should be undefined. |
| + shouldBe("document.pageDefinedVar", "undefined"); |
|
haraken
2013/06/27 04:50:27
Ditto.
jww
2013/06/27 17:44:42
Done.
|
| + |
| + window.postMessage("done", "*"); |
| + }); |
| +} |
| + |
| +function sendDocumentObject(eventType, prop) { |
| + var newEvent = eval("new " + eventType + "('" + eventType + "', { " + prop + ": document })"); |
| + document.dispatchEvent(newEvent); |
| +} |
| + |
| +function runScript(eventType, prop) { |
| + // Final string should have the form: |
| + // document.pageDefinedVar = 1; (function sendDocumentObject(eventType, prop) {...})(...); |
| + // When evaluated in the isolated world, should initiate the event with the |
| + // document object as the specificed property value. |
| + var script = "document.pageDefinedVar = 1; " + |
| + "(" + sendDocumentObject.toString() + ")('" + eventType + "', '" + prop + "');"; |
| + addListener(eventType, prop); |
| + testRunner.evaluateScriptInIsolatedWorld(1, script); |
| +} |
| + |
| +// Run the tests whenever a notification arrives, which indicates that the |
| +// previous test has finished. |
| +window.addEventListener("message", function(message) { |
| + runNextTest(); |
| +}, false); |
| + |
| +// The events that we want to test, with the properties that each one uses. |
| +var events = [ |
| + { eventType: "CustomEvent", prop: "detail" }, |
| + { eventType: "MessageEvent", prop: "data" }, |
| + { eventType: "PopStateEvent", prop: "state" } |
| +]; |
| + |
| +function runNextTest () { |
| + var evt = events.pop(); |
| + if (!evt) { |
| + finishJSTest(); |
| + return; |
| + } |
| + |
| + runScript(evt.eventType, evt.prop); |
| +}; |
| + |
| +// This test is meaningless without testRunner. |
| +if (window.testRunner) { |
| + runNextTest(); |
| +} |
| +</script> |
| +</body> |
| +<script src="../js/resources/js-test-post.js"></script> |
| +</html> |
