Bug 1507837 [wpt PR 14094] - Remove timeout in async_test for miscellaneous tests, a=testonly
authorqiuzhong <zhongx.qiu@intel.com>
Fri, 30 Nov 2018 16:54:14 +0000
changeset 449820 456273bb9289767db169fe79104ba8320e45cabe
parent 449819 9caf650605472e0569cd50ab1d64bd557de4c163
child 449821 9acf8f2e2e828b5ba2d37fb4d4150db6d6808b02
push id35187
push userncsoregi@mozilla.com
push dateTue, 11 Dec 2018 16:25:40 +0000
treeherdermozilla-central@a02122d22c5e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1507837, 14094
milestone66.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1507837 [wpt PR 14094] - Remove timeout in async_test for miscellaneous tests, a=testonly Automatic update from web-platform-testsRemove timeout in async_test for miscellaneous tests (#14094) Remove all the timeout in `async_test` for miscellaneous tests except html, IndexedDB, websockets, mediacapture-streams and cookies. -- wpt-commits: 565cc2b9a470fdf83c703033d7beb3140aa3a6ce wpt-pr: 14094
testing/web-platform/tests/cors/preflight-cache.htm
testing/web-platform/tests/cors/status-async.htm
testing/web-platform/tests/css/css-backgrounds/border-image-repeat_repeatnegx_none_50px.html
testing/web-platform/tests/css/css-flexbox/display_flex_exist.html
testing/web-platform/tests/css/css-flexbox/display_inline-flex_exist.html
testing/web-platform/tests/css/css-flexbox/order_value.html
testing/web-platform/tests/custom-elements/CustomElementRegistry.html
testing/web-platform/tests/eventsource/dedicated-worker/eventsource-constructor-non-same-origin.htm
testing/web-platform/tests/eventsource/eventsource-close.htm
testing/web-platform/tests/eventsource/eventsource-constructor-non-same-origin.htm
testing/web-platform/tests/eventsource/eventsource-reconnect.htm
testing/web-platform/tests/eventsource/shared-worker/eventsource-constructor-non-same-origin.htm
testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/html-forms/test-003.html
testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/inert-html-elements/test-001.html
--- a/testing/web-platform/tests/cors/preflight-cache.htm
+++ b/testing/web-platform/tests/cors/preflight-cache.htm
@@ -73,17 +73,17 @@ test(function() {
     var client = new XMLHttpRequest()
 
     var id = did_preflight(true, client, {extra:'max_age=-1'})
     did_preflight(true, client, {extra:'max_age=-1', token: id})
 },
 'age = -1, should not be cached');
 
 (function() {
-    var test = async_test("preflight first request, second from cache, wait, third should preflight again", { timeout: 6000 }),
+    var test = async_test("preflight first request, second from cache, wait, third should preflight again"),
         time = new Date().getTime(),
         dothing = function (url, msg, set_request, func) {
             client = new XMLHttpRequest(),
             client.open('GET', url, true)
             if (set_request)
                 client.setRequestHeader('x-print', msg)
             client.onload = test.step_func(function() {
                 assert_equals(client.response, msg, "response " + url)
--- a/testing/web-platform/tests/cors/status-async.htm
+++ b/testing/web-platform/tests/cors/status-async.htm
@@ -9,17 +9,17 @@
 <script src=support.js?pipe=sub></script>
 
 <h1>Status returned</h1>
 
 <div id=log></div>
 <script>
 
 function statusRequest(method, code, text, content, type) {
-    async_test("Status on " + method + " " + code, { timeout: 15000 })
+    async_test("Status on " + method + " " + code)
     .step(function() {
         var client = new XMLHttpRequest()
         client.open(method, CROSSDOMAIN + "resources/status.py?code="
             + code + "&text=" + text + "&content=" + content + "&type=" + type, true)
         client.onreadystatechange = this.step_func(function() {
             if (client.readyState != client.DONE)
                 return
 
@@ -55,17 +55,17 @@ function statusRequest(method, code, tex
   statusRequest("PUT",     200, 'OK', '400',         'text/plain')
   statusRequest("CHICKEN", 200, 'OK', 'bah',         '')
 
 
 function statusRequestFail(method, code, expect_code, nonsimple) {
     if (expect_code === undefined)
         expect_code = code
 
-    async_test("Status on " + method + " " + code + (nonsimple?' (nonsimple)':''), { timeout: 15000 })
+    async_test("Status on " + method + " " + code + (nonsimple?' (nonsimple)':''))
     .step(function() {
         var client = new XMLHttpRequest()
 
         client.open(method, CROSSDOMAIN + "resources/status.py?code="
           + code + '&headers=x-nonsimple&text=OHAI', true)
 
         if (nonsimple)
             client.setRequestHeader('x-nonsimple', true)
--- a/testing/web-platform/tests/css/css-backgrounds/border-image-repeat_repeatnegx_none_50px.html
+++ b/testing/web-platform/tests/css/css-backgrounds/border-image-repeat_repeatnegx_none_50px.html
@@ -17,17 +17,17 @@
         }
     </style>
 </head>
 <body>
     <div id="log"></div>
     <div id="test"></div>
     <script type="text/javascript">
         var div = document.querySelector("#test");
-        var t = async_test(document.title, { timeout: 500 });
+        var t = async_test(document.title);
         t.step(function () {
             div.style[headProp("border-image-repeat")] = "repeat-x";
             div.style[headProp("height")] = "200px";
             div.style[headProp("width")] = "200px";
             div.style[headProp("border-image-source")] = "none";
             div.style[headProp("border-image-width")] = "50px";
             var propvalue = GetCurrentStyle("border-image-repeat");
             var prop = propvalue.indexOf("repeat-x")!=-1;
--- a/testing/web-platform/tests/css/css-flexbox/display_flex_exist.html
+++ b/testing/web-platform/tests/css/css-flexbox/display_flex_exist.html
@@ -13,16 +13,16 @@
     </style>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <div id=log></div>
     <div id=test></div>
     <script type="text/javascript">
-        var t = async_test(document.title, {timeout:500});
+        var t = async_test(document.title);
         t.step(function () {
             assert_equals(window.getComputedStyle(document.querySelector('div#test')).display, "flex", "Display value is");
         });
         t.done();
     </script>
   </body>
 </html>
--- a/testing/web-platform/tests/css/css-flexbox/display_inline-flex_exist.html
+++ b/testing/web-platform/tests/css/css-flexbox/display_inline-flex_exist.html
@@ -13,16 +13,16 @@
     </style>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <div id=log></div>
     <div id=test></div>
     <script type="text/javascript">
-        var t = async_test(document.title, {timeout:500});
+        var t = async_test(document.title);
         t.step(function () {
             assert_equals(window.getComputedStyle(document.querySelector('div#test')).display, "inline-flex", "Display value is");
         });
         t.done();
     </script>
   </body>
 </html>
--- a/testing/web-platform/tests/css/css-flexbox/order_value.html
+++ b/testing/web-platform/tests/css/css-flexbox/order_value.html
@@ -19,17 +19,17 @@
     </style>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <div id=log></div>
     <div id=test><div id=test01>1</div><div id=test02>2</div><div id=test03>3</div></div>
     <script type="text/javascript">
-        var t = async_test(document.title, {timeout:500});
+        var t = async_test(document.title);
         t.step(function () {
             assert_equals(document.getElementById("test01").offsetTop, document.getElementById("test02").offsetTop, "Rectangle 1 and 2 have the same offsetTop value");
             assert_equals((document.getElementById("test02").offsetLeft >= document.getElementById("test01").offsetLeft), false, "Rectangle 2 have a smaller offsetLeft value than 1.");
         });
         t.done();
     </script>
   </body>
 </html>
--- a/testing/web-platform/tests/custom-elements/CustomElementRegistry.html
+++ b/testing/web-platform/tests/custom-elements/CustomElementRegistry.html
@@ -170,17 +170,17 @@ test(function () {
     assert_false(unresolvedElement instanceof ConstructorCallsDefine);
     assert_false(elementUpgradedDuringUpgrade instanceof DefinedDuringUpgrade);
 
     customElements.define('constructor-calls-define', ConstructorCallsDefine);
 }, 'customElements.define unset the element definition is running flag before upgrading custom elements');
 
 (function () {
     var testCase = async_test('customElements.define must not throw'
-        +' when defining another custom element in a different global object during Get(constructor, "prototype")', {timeout: 100});
+        +' when defining another custom element in a different global object during Get(constructor, "prototype")');
 
     var iframe = document.createElement('iframe');
     iframe.onload = function () {
         testCase.step(function () {
             var InnerCustomElement = class extends iframe.contentWindow.HTMLElement {};
             var calls = [];
             var proxy = new Proxy(class extends HTMLElement { }, {
                 get: function (target, name) {
--- a/testing/web-platform/tests/eventsource/dedicated-worker/eventsource-constructor-non-same-origin.htm
+++ b/testing/web-platform/tests/eventsource/dedicated-worker/eventsource-constructor-non-same-origin.htm
@@ -17,17 +17,17 @@ try {
     <meta name=timeout content=long>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <div id="log"></div>
     <script>
       function fetchFail(url) {
-        var test = async_test(document.title + " (" + url + ")", { timeout: 20000 })
+        var test = async_test(document.title + " (" + url + ")")
         test.step(function() {
           var worker = new Worker('#'+encodeURIComponent(url))
           worker.onmessage = function(e) {
             test.step(function() {
               assert_true(e.data[0], e.data[1])
               assert_equals(e.data[1], EventSource.CLOSED, 'source.readyState')
               assert_false(e.data[2], "'data' in e");
             })
--- a/testing/web-platform/tests/eventsource/eventsource-close.htm
+++ b/testing/web-platform/tests/eventsource/eventsource-close.htm
@@ -15,17 +15,17 @@
         source.onopen = this.step_func(function() {
           assert_equals(source.readyState, source.OPEN, "open readyState");
           source.close()
           assert_equals(source.readyState, source.CLOSED, "closed readyState");
           this.done()
         })
       })
 
-      var test2 = async_test(document.title + ", test events", { timeout: 9000 });
+      var test2 = async_test(document.title + ", test events");
       test2.step(function() {
         var count = 0, reconnected = false,
             source = new EventSource("resources/reconnect-fail.py?id=" + new Date().getTime());
 
         source.onerror = this.step_func(function(e) {
           assert_equals(e.type, 'error');
           switch(count) {
             // reconnecting after first message
--- a/testing/web-platform/tests/eventsource/eventsource-constructor-non-same-origin.htm
+++ b/testing/web-platform/tests/eventsource/eventsource-constructor-non-same-origin.htm
@@ -4,17 +4,17 @@
     <title>EventSource: constructor (act as if there is a network error)</title>
     <meta name=timeout content=long>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <script>
       function fetchFail(url) {
-        var test = async_test(document.title + " (" + url + ")", { timeout: 20000 })
+        var test = async_test(document.title + " (" + url + ")")
         test.step(function() {
           var source = new EventSource(url)
           source.onerror = function(e) {
             test.step(function() {
               assert_equals(source.readyState, source.CLOSED)
               assert_false(e.hasOwnProperty('data'))
             })
             test.done()
--- a/testing/web-platform/tests/eventsource/eventsource-reconnect.htm
+++ b/testing/web-platform/tests/eventsource/eventsource-reconnect.htm
@@ -19,17 +19,17 @@
           })
         })
       }
 
       doReconn("resources/status-reconnect.py?status=200",
         "200")
 
 
-      var t = async_test(document.title + ", test reconnection events", { timeout: 9000 });
+      var t = async_test(document.title + ", test reconnection events");
       t.step(function() {
         var opened = false, reconnected = false,
             source = new EventSource("resources/status-reconnect.py?status=200&ok_first&id=2");
 
         source.onerror = t.step_func(function(e) {
           assert_equals(e.type, 'error');
           assert_equals(source.readyState, source.CONNECTING, "readyState");
           assert_true(opened, "connection is opened earlier");
--- a/testing/web-platform/tests/eventsource/shared-worker/eventsource-constructor-non-same-origin.htm
+++ b/testing/web-platform/tests/eventsource/shared-worker/eventsource-constructor-non-same-origin.htm
@@ -20,17 +20,17 @@ try {
     <meta name=timeout content=long>
     <script src="/resources/testharness.js"></script>
     <script src="/resources/testharnessreport.js"></script>
   </head>
   <body>
     <div id="log"></div>
     <script>
       function fetchFail(url) {
-        var test = async_test(document.title + " (" + url + ")", { timeout: 20000 })
+        var test = async_test(document.title + " (" + url + ")")
         test.step(function() {
           var worker = new SharedWorker('#'+encodeURIComponent(url))
           worker.port.onmessage = function(e) {
             test.step(function() {
               assert_true(e.data[0], e.data[1])
               assert_equals(e.data[1], EventSource.CLOSED, 'source.readyState')
               assert_false(e.data[2], "'data' in e");
             })
--- a/testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/html-forms/test-003.html
+++ b/testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/html-forms/test-003.html
@@ -16,17 +16,17 @@ policies and contribution forms [3].
 <meta name="assert" content="HTML Elements in shadow trees: form should not submit elements in shadow tree">
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 <script src="../../../../html/resources/common.js"></script>
 </head>
 <body>
 <div id="log"></div>
 <script>
-var A_08_02_03_T01 = async_test('A_08_02_03_T01', { timeout: 5000 });
+var A_08_02_03_T01 = async_test('A_08_02_03_T01');
 
 var checkIframeContent = A_08_02_03_T01.step_func(function () {
     // remember value to check before cleaning the context (it'll destroy the iframe)
     var valueToCheck = A_08_02_03_T01.iframe.contentWindow.document.URL;
     cleanContext(A_08_02_03_T01.ctx);
 
     assert_true(valueToCheck.indexOf('inp1=value1') > 0,
         'html form should submit all of its fields');
--- a/testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/inert-html-elements/test-001.html
+++ b/testing/web-platform/tests/shadow-dom/untriaged/html-elements-in-shadow-trees/inert-html-elements/test-001.html
@@ -16,17 +16,17 @@ policies and contribution forms [3].
 <meta name="assert" content="HTML Elements in shadow trees: base element must behave as inert, or not part of the document tree">
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
 <script src="../../../../html/resources/common.js"></script>
 </head>
 <body>
 <div id="log"></div>
 <script>
-var A_08_01_01_T01 = async_test('A_08_01_01_T01', { timeout: 5000 });
+var A_08_01_01_T01 = async_test('A_08_01_01_T01');
 
 A_08_01_01_T01.checkIframeContent = A_08_01_01_T01.step_func(function () {
     //remember value to check before cleaning the context (it'll destroy the iframe)
     var valueToCheck = A_08_01_01_T01.iframe.contentWindow;
     cleanContext(A_08_01_01_T01.ctx);
 
     assert_equals(valueToCheck, null,
         'base html element ih a shadow tree must beahve like inert one');