Test that beforescriptexecute/afterscriptexecute are not supported (#…

…3687); r=Ms2ger See whatwg/html#943
web-platform-tests · Sep 9, 2016 · d0411c3 · d0411c3
1 parent b2c070b
commit d0411c3
Show file tree

Hide file tree

Showing 3 changed files with 53 additions and 75 deletions.
diff --git a/html/dom/documents/dom-tree-accessors/Document.currentScript.sub.html b/html/dom/documents/dom-tree-accessors/Document.currentScript.sub.html
@@ -102,26 +102,6 @@
  finish('nested');
 </script>
 
-<!-- Test beforescriptexecute and afterscriptexecute -->
-<script id="script-exec-before-after">
-function verifyScriptExec(e) {
-  verify('script-exec');
-}
-
-document.addEventListener('beforescriptexecute', verifyScriptExec, false);
-document.addEventListener('afterscriptexecute', verifyScriptExec, false);
-
-var s = document.createElement("script");
-s.id = "script-exec-test";
-s.textContent = "function nop() { return false }";
-document.body.appendChild(s);
-
-document.removeEventListener('beforescriptexecute', verifyScriptExec);
-document.removeEventListener('afterscriptexecute', verifyScriptExec);
-
-finish('script-exec');
-</script>
-
 <!-- Test script load error event listener -->
 <script>
 function testLoadFail() {

diff --git a/html/semantics/scripting-1/the-script-element/historical.html b/html/semantics/scripting-1/the-script-element/historical.html
@@ -0,0 +1,53 @@
+<!doctype html>
+<title>Historical script element features should not be supported</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script>
+function t(property) {
+  test(function() {
+    assert_false(property in document.createElement('script'));
+  }, 'script.' + property + ' should not be supported');
+}
+// added in https://github.com/whatwg/html/commit/69f83cf2eacf4543860ccb7abab0ff5bb1e8b594
+// removed in https://github.com/whatwg/html/commit/1a0b5e8377d59462e05a5cffda4b8592324a2785
+t('onbeforescriptexecute');
+t('onafterscriptexecute');
+
+var t_onbeforescriptexecute_attr = async_test('onbeforescriptexecute content attribute should not be supported');
+var t_onafterscriptexecute_attr = async_test('onafterscriptexecute content attribute should not be supported');
+var t_beforescriptexecute_event = async_test(function() {
+  addEventListener('beforescriptexecute', this.step_func(function() {
+    assert_unreached();
+  }), true);
+}, 'beforescriptexecute event should not be supported');
+var t_afterscriptexecute_event = async_test(function() {
+  addEventListener('afterscriptexecute', this.step_func(function() {
+    assert_unreached();
+  }), true);
+}, 'afterscriptexecute event should not be supported');
+
+var a = false;
+
+onload = function() {
+  t_onbeforescriptexecute_attr.step(function() {
+    assert_true(a);
+    this.done();
+  });
+  t_onafterscriptexecute_attr.step(function() {
+    assert_true(a);
+    this.done();
+  });
+  t_beforescriptexecute_event.step(function() {
+    assert_true(a);
+    this.done();
+  });
+  t_afterscriptexecute_event.step(function() {
+    assert_true(a);
+    this.done();
+  });
+};
+</script>
+<script onbeforescriptexecute="t_onbeforescriptexecute_attr.step(function() { assert_unreached(); });"
+        onafterscriptexecute="t_onafterscriptexecute_attr.step(function() { assert_unreached(); });">
+a = true;
+</script>
diff --git a/html/semantics/scripting-1/the-script-element/script-before-after-events.html b/html/semantics/scripting-1/the-script-element/script-before-after-events.html