Chromium Code Reviews Chromium Code Reviews
Issue 2145893003:
Remove video-test.js dependency from video-autoplay-experiment-just-once.html (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html> | |
| 1 <script src=media-file.js></script> | 2 <script src=media-file.js></script> |
| 2 <!-- TODO(foolip): Convert test to testharness.js. crbug.com/588956 | |
| 3 (Please avoid writing new tests using video-test.js) --> | |
| 4 <script src=video-test.js></script> | 3 <script src=video-test.js></script> |
|
fs
2016/07/13 14:46:34
I think eliminating video-test.js stuff would stil
Srirama
2016/07/13 15:35:29
Right. Removed it.
| |
| 5 <script> | 4 <script> |
| 6 | 5 |
| 7 function runTest() | 6 function runTest() |
| 8 { | 7 { |
| 9 var canPlayThroughCount = 0; | 8 var canPlayThroughCount = 0; |
| 10 var videoShouldPlay; | 9 var videoShouldPlay; |
| 11 var videoShouldNotPlay; | 10 var videoShouldNotPlay; |
| 12 | 11 |
| 13 testRunner.waitUntilDone(); | 12 testRunner.waitUntilDone(); |
| 14 | 13 |
| (...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 64 | 63 |
| 65 videoShouldPlay = prepareVideo(parent); | 64 videoShouldPlay = prepareVideo(parent); |
| 66 videoShouldNotPlay = prepareVideo(parent); | 65 videoShouldNotPlay = prepareVideo(parent); |
| 67 } | 66 } |
| 68 | 67 |
| 69 </script> | 68 </script> |
| 70 <p>Test that the autoplay experiment doesn't play media once the media | 69 <p>Test that the autoplay experiment doesn't play media once the media |
| 71 is no longer eligible for autoplay.</p> | 70 is no longer eligible for autoplay.</p> |
| 72 <body onLoad="runTest()"> | 71 <body onLoad="runTest()"> |
| 73 </body> | 72 </body> |
| OLD | NEW |
