From cb656d1138eb6b22db5d0f9db10bd64ccd9bf1ed Mon Sep 17 00:00:00 2001 From: qiuzhong Date: Thu, 15 Nov 2018 15:40:26 +0800 Subject: [PATCH 1/2] Remove timeout in async_test for html tests Remove all the timeout in `async_test` for html tests. Affected tests: 125 Before: Pass: 64, Failed: 61 After: Pass: 64, Failed: 61 Related: #11120 --- .../browsing_context_name_cross_origin_2.html | 2 +- .../browsing_context_name_cross_origin_3.html | 2 +- .../browsing-the-web/navigating-across-documents/010.html | 2 +- .../browsing-the-web/navigating-across-documents/012.html | 2 +- .../browsing-the-web/navigating-across-documents/013.html | 2 +- .../browsing-the-web/navigating-across-documents/014.html | 2 +- .../browsing-the-web/navigating-across-documents/015.html | 2 +- .../javascript-url-query-fragment-components.html | 2 +- .../history/the-history-interface/history_back_1.html | 2 +- .../history/the-history-interface/history_forward_1.html | 2 +- .../the-history-interface/history_go_no_argument.html | 2 +- .../history/the-history-interface/history_go_to_uri.html | 2 +- .../the-history-interface/history_go_undefined.html | 2 +- .../history/the-history-interface/history_go_zero.html | 2 +- .../traverse_the_history_unload_prompt_1-manual.html | 2 +- .../traverse_the_history_unload_prompt_2-manual.html | 2 +- ...averse_the_session_history_unload_prompt_1-manual.html | 2 +- .../the-history-interface/traverse_the_history_3.html | 2 +- .../the-history-interface/traverse_the_history_4.html | 2 +- .../the-history-interface/traverse_the_history_5.html | 2 +- .../traverse_the_history_unload_1.html | 2 +- .../manual_click_assign_during_load-manual.html | 2 +- .../manual_click_location_replace_during_load-manual.html | 2 +- .../manual_click_replace_during_load-manual.html | 2 +- .../manual_form_submit_assign_during_load-manual.html | 2 +- .../non-automated/reload_in_resize-manual.html | 2 +- .../scripted_click_assign_during_load.html | 2 +- .../scripted_form_submit_assign_during_load.html | 2 +- .../close_script_defer.html | 2 +- .../non-automated/discard_iframe_history_1-manual.html | 2 +- .../non-automated/discard_iframe_history_2-manual.html | 2 +- html/interaction/focus/focus-01.html | 2 +- .../focus-tabindex-order.html | 2 +- .../embedded-content/media-elements/event_canplay.html | 4 ++-- .../media-elements/event_canplay_noautoplay.html | 4 ++-- .../media-elements/event_canplaythrough.html | 4 ++-- .../media-elements/event_canplaythrough_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_loadeddata.html | 4 ++-- .../media-elements/event_loadeddata_noautoplay.html | 4 ++-- .../media-elements/event_loadedmetadata.html | 4 ++-- .../media-elements/event_loadedmetadata_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_loadstart.html | 4 ++-- .../media-elements/event_loadstart_noautoplay.html | 4 ++-- .../event_order_canplay_canplaythrough.html | 4 ++-- .../media-elements/event_order_canplay_playing.html | 4 ++-- .../event_order_loadedmetadata_loadeddata.html | 4 ++-- .../media-elements/event_order_loadstart_progress.html | 4 ++-- .../embedded-content/media-elements/event_pause.html | 4 ++-- .../media-elements/event_pause_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_play.html | 4 ++-- .../media-elements/event_play_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_playing.html | 4 ++-- .../media-elements/event_playing_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_progress.html | 4 ++-- .../media-elements/event_progress_noautoplay.html | 4 ++-- .../embedded-content/media-elements/event_timeupdate.html | 4 ++-- .../media-elements/event_timeupdate_noautoplay.html | 4 ++-- .../media-elements/interfaces/TextTrack/activeCues.html | 8 ++++---- .../media-elements/interfaces/TextTrack/addCue.html | 2 +- .../media-elements/interfaces/TextTrack/removeCue.html | 2 +- .../media-elements/networkState_during_loadstart.html | 4 ++-- .../media-elements/networkState_during_progress.html | 4 ++-- .../media-elements/paused_false_during_play.html | 4 ++-- .../media-elements/paused_true_during_pause.html | 4 ++-- .../media-elements/readyState_during_canplay.html | 4 ++-- .../media-elements/readyState_during_canplaythrough.html | 4 ++-- .../media-elements/readyState_during_loadeddata.html | 4 ++-- .../media-elements/readyState_during_loadedmetadata.html | 4 ++-- .../media-elements/readyState_during_playing.html | 4 ++-- .../track/track-element/cors/support/common.js | 2 +- .../the-script-element/execution-timing/078.html | 2 +- .../the-script-element/execution-timing/081.html | 2 +- .../the-script-element/execution-timing/082.html | 2 +- .../the-script-element/execution-timing/091.html | 2 +- .../the-script-element/execution-timing/092.html | 2 +- .../the-script-element/execution-timing/094.html | 2 +- .../the-script-element/execution-timing/095.html | 2 +- .../the-script-element/execution-timing/097.html | 2 +- .../the-script-element/execution-timing/122.html | 2 +- .../the-script-element/execution-timing/123.html | 2 +- .../the-script-element/execution-timing/125.html | 2 +- .../the-script-element/execution-timing/126.html | 2 +- .../the-script-element/execution-timing/136.html | 2 +- .../the-script-element/execution-timing/137.html | 2 +- .../the-script-element/execution-timing/138.html | 2 +- .../the-script-element/execution-timing/139.html | 2 +- .../the-script-element/execution-timing/140.html | 2 +- .../the-script-element/execution-timing/141.html | 2 +- .../the-script-element/execution-timing/142.html | 2 +- .../the-script-element/execution-timing/143.html | 2 +- .../the-script-element/execution-timing/144.html | 2 +- .../the-script-element/execution-timing/145.html | 2 +- 92 files changed, 128 insertions(+), 128 deletions(-) diff --git a/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html b/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html index f1be0439df3670..1b9fee04fbfb38 100644 --- a/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html +++ b/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html @@ -6,7 +6,7 @@

 
 '); return ''})()">
 Test
 '); return ''})()" href="href.html">Test
 
 
diff --git a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html index 27d395d22c46bd..8a8904574e23bc 100644 --- a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html +++ b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html @@ -5,6 +5,6 @@
diff --git a/html/interaction/focus/focus-01.html b/html/interaction/focus/focus-01.html index 9d1bf1b6c7d3f3..b3696e9794ed29 100644 --- a/html/interaction/focus/focus-01.html +++ b/html/interaction/focus/focus-01.html @@ -21,7 +21,7 @@ setup(function () { testEle = document.getElementById("test"); testEle.focus(); -}, {timeout: 10000}); +}); document.onkeydown = t1.step_func_done(function(evt){ assert_equals(evt.target, testEle, "The keydown events must be targeted at the input element."); diff --git a/html/interaction/focus/sequential-focus-navigation-and-the-tabindex-attribute/focus-tabindex-order.html b/html/interaction/focus/sequential-focus-navigation-and-the-tabindex-attribute/focus-tabindex-order.html index 45429cc1a72c65..eadd95aac52763 100644 --- a/html/interaction/focus/sequential-focus-navigation-and-the-tabindex-attribute/focus-tabindex-order.html +++ b/html/interaction/focus/sequential-focus-navigation-and-the-tabindex-attribute/focus-tabindex-order.html @@ -31,7 +31,7 @@ setup(function () { document.body.focus(); -}, {timeout: 20000}); +}); diff --git a/html/semantics/embedded-content/media-elements/event_canplay.html b/html/semantics/embedded-content/media-elements/event_canplay.html index e79c5d1c76dc09..e5c632bc17bb8d 100644 --- a/html/semantics/embedded-content/media-elements/event_canplay.html +++ b/html/semantics/embedded-content/media-elements/event_canplay.html @@ -15,7 +15,7 @@
From 59a1292ed10b27eb7f3b6e25017aaac31ce26b1b Mon Sep 17 00:00:00 2001 From: qiuzhong Date: Wed, 21 Nov 2018 11:20:10 +0800 Subject: [PATCH 2/2] Remove unnecessary undefined parameter for async_test After remove the timeout parameter from async_test, the first parameter `undefined` is no use and should be removed. --- .../history-traversal/browsing_context_name_cross_origin_2.html | 2 +- .../history-traversal/browsing_context_name_cross_origin_3.html | 2 +- .../browsing-the-web/navigating-across-documents/010.html | 2 +- .../browsing-the-web/navigating-across-documents/012.html | 2 +- .../browsing-the-web/navigating-across-documents/013.html | 2 +- .../browsing-the-web/navigating-across-documents/014.html | 2 +- .../browsing-the-web/navigating-across-documents/015.html | 2 +- html/browsers/history/the-history-interface/history_back_1.html | 2 +- .../history/the-history-interface/history_forward_1.html | 2 +- .../history/the-history-interface/history_go_no_argument.html | 2 +- .../history/the-history-interface/history_go_to_uri.html | 2 +- .../history/the-history-interface/history_go_undefined.html | 2 +- .../browsers/history/the-history-interface/history_go_zero.html | 2 +- .../traverse_the_history_unload_prompt_1-manual.html | 2 +- .../traverse_the_history_unload_prompt_2-manual.html | 2 +- .../traverse_the_session_history_unload_prompt_1-manual.html | 2 +- .../history/the-history-interface/traverse_the_history_3.html | 2 +- .../history/the-history-interface/traverse_the_history_4.html | 2 +- .../history/the-history-interface/traverse_the_history_5.html | 2 +- .../the-history-interface/traverse_the_history_unload_1.html | 2 +- .../non-automated/manual_click_assign_during_load-manual.html | 2 +- .../manual_click_location_replace_during_load-manual.html | 2 +- .../non-automated/manual_click_replace_during_load-manual.html | 2 +- .../manual_form_submit_assign_during_load-manual.html | 2 +- .../non-automated/reload_in_resize-manual.html | 2 +- .../scripted_click_assign_during_load.html | 2 +- .../scripted_form_submit_assign_during_load.html | 2 +- .../close_script_defer.html | 2 +- .../discard_iframe_history_3.html | 2 +- .../non-automated/discard_iframe_history_1-manual.html | 2 +- .../non-automated/discard_iframe_history_2-manual.html | 2 +- .../scripting-1/the-script-element/execution-timing/078.html | 2 +- .../scripting-1/the-script-element/execution-timing/081.html | 2 +- .../scripting-1/the-script-element/execution-timing/082.html | 2 +- .../scripting-1/the-script-element/execution-timing/091.html | 2 +- .../scripting-1/the-script-element/execution-timing/092.html | 2 +- .../scripting-1/the-script-element/execution-timing/094.html | 2 +- .../scripting-1/the-script-element/execution-timing/095.html | 2 +- .../scripting-1/the-script-element/execution-timing/097.html | 2 +- .../scripting-1/the-script-element/execution-timing/122.html | 2 +- .../scripting-1/the-script-element/execution-timing/123.html | 2 +- .../scripting-1/the-script-element/execution-timing/125.html | 2 +- .../scripting-1/the-script-element/execution-timing/126.html | 2 +- .../scripting-1/the-script-element/execution-timing/136.html | 2 +- .../scripting-1/the-script-element/execution-timing/137.html | 2 +- .../scripting-1/the-script-element/execution-timing/138.html | 2 +- .../scripting-1/the-script-element/execution-timing/139.html | 2 +- .../scripting-1/the-script-element/execution-timing/140.html | 2 +- .../scripting-1/the-script-element/execution-timing/141.html | 2 +- .../scripting-1/the-script-element/execution-timing/142.html | 2 +- .../scripting-1/the-script-element/execution-timing/143.html | 2 +- .../scripting-1/the-script-element/execution-timing/144.html | 2 +- .../scripting-1/the-script-element/execution-timing/145.html | 2 +- 53 files changed, 53 insertions(+), 53 deletions(-) diff --git a/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html b/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html index 1b9fee04fbfb38..8202a892a34908 100644 --- a/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html +++ b/html/browsers/browsing-the-web/history-traversal/browsing_context_name_cross_origin_2.html @@ -6,7 +6,7 @@

 
 '); return ''})()">
 Test
 '); return ''})()" href="href.html">Test
 
 
diff --git a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_1-manual.html b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_1-manual.html index 3dbad6f2947f5f..ca9dd91f2e16a6 100644 --- a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_1-manual.html +++ b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_1-manual.html @@ -5,6 +5,6 @@
diff --git a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html index 8a8904574e23bc..57eaabcd0cea1d 100644 --- a/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html +++ b/html/browsers/the-window-object/garbage-collection-and-browsing-contexts/non-automated/discard_iframe_history_2-manual.html @@ -5,6 +5,6 @@
diff --git a/html/semantics/scripting-1/the-script-element/execution-timing/078.html b/html/semantics/scripting-1/the-script-element/execution-timing/078.html index 274fb9ea344cc1..644c543611ecca 100644 --- a/html/semantics/scripting-1/the-script-element/execution-timing/078.html +++ b/html/semantics/scripting-1/the-script-element/execution-timing/078.html @@ -15,7 +15,7 @@ } return testlib.addScript(contents, props, head, false); } - var t = async_test(undefined) + var t = async_test() function test() { document.getElementById("log").textContent = "Please wait..." diff --git a/html/semantics/scripting-1/the-script-element/execution-timing/081.html b/html/semantics/scripting-1/the-script-element/execution-timing/081.html index 5e31defca8abd5..1b9bc991c69355 100644 --- a/html/semantics/scripting-1/the-script-element/execution-timing/081.html +++ b/html/semantics/scripting-1/the-script-element/execution-timing/081.html @@ -10,7 +10,7 @@
FAILED (This TC requires JavaScript enabled)