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

Side by Side Diff: third_party/WebKit/LayoutTests/media/audio-garbage-collect.html

Issue 1715303002: Add TODOs to convert from video-test.js to testharness.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add bug ref Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 <!DOCTYPE HTML> 1 <!DOCTYPE HTML>
2 2
3 <html> 3 <html>
4 <body> 4 <body>
5 5
6 <p>Tests that we don't garbage collect playing audio object or event listener.</ p> 6 <p>Tests that we don't garbage collect playing audio object or event listener.</ p>
7 <p>According to http://www.whatwg.org/specs/web-apps/current-work/multipage/the- video-element.html,<br /> 7 <p>According to http://www.whatwg.org/specs/web-apps/current-work/multipage/the- video-element.html,<br />
8 "4.8.10.8 Playing the media resource",<br /> 8 "4.8.10.8 Playing the media resource",<br />
9 "Media elements must not stop playing just because all references to them have 9 "Media elements must not stop playing just because all references to them have
10 been removed; only once a media element is in a state where no further audio 10 been removed; only once a media element is in a state where no further audio
11 could ever be played by that element may the element be garbage collected."<br / ><br /> 11 could ever be played by that element may the element be garbage collected."<br / ><br />
12 (see https://bugs.webkit.org/show_bug.cgi?id=66878, https://bugs.webkit.org/show _bug.cgi?id=70421, and http://crbug.com/62604 for more details).</p> 12 (see https://bugs.webkit.org/show_bug.cgi?id=66878, https://bugs.webkit.org/show _bug.cgi?id=70421, and http://crbug.com/62604 for more details).</p>
13 <p id="result"> 13 <p id="result">
14 FAIL: Test either still running or stopped prematurely. 14 FAIL: Test either still running or stopped prematurely.
15 </p> 15 </p>
16 16
17 <script src=../resources/gc.js></script> 17 <script src=../resources/gc.js></script>
18 <script src=media-file.js></script> 18 <script src=media-file.js></script>
19 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
20 (Please avoid writing new tests using video-test.js) -->
19 <script src=video-test.js></script> 21 <script src=video-test.js></script>
20 <script type="text/javascript"> 22 <script type="text/javascript">
21 23
22 var num_players = 4; 24 var num_players = 4;
23 var play_times = 5; 25 var play_times = 5;
24 26
25 function finish() { 27 function finish() {
26 document.getElementById("result").innerText = "PASS"; 28 document.getElementById("result").innerText = "PASS";
27 if (window.testRunner) { 29 if (window.testRunner) {
28 testRunner.notifyDone(); 30 testRunner.notifyDone();
(...skipping 26 matching lines...) Expand all
55 a.currentTime = a.duration - 0.35; 57 a.currentTime = a.duration - 0.35;
56 a.play(); 58 a.play();
57 }, false, true, a, true); 59 }, false, true, a, true);
58 } 60 }
59 61
60 start(); 62 start();
61 63
62 </script> 64 </script>
63 </body> 65 </body>
64 </html> 66 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698