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

Side by Side Diff: third_party/WebKit/LayoutTests/media/video-autoplay-experiment-modes.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, 10 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 <html> 1 <html>
2 <video autoplay controls></video> 2 <video autoplay controls></video>
3 <script src=media-file.js></script> 3 <script src=media-file.js></script>
4 <!-- TODO(philipj): Convert test to testharness.js. crbug.com/588956
5 (Please avoid writing new tests using video-test.js) -->
4 <script src=video-test.js></script> 6 <script src=video-test.js></script>
5 <body> 7 <body>
6 <pre> 8 <pre>
7 Check if the autoplay gesture override experiment works. There are a lot 9 Check if the autoplay gesture override experiment works. There are a lot
8 of config options, so this test just runs all of them. 10 of config options, so this test just runs all of them.
9 11
10 The "results" table contains one row per config tested. 12 The "results" table contains one row per config tested.
11 == Test Inputs == 13 == Test Inputs ==
12 # - config number, in case you'd like to run just one. 14 # - config number, in case you'd like to run just one.
13 Flags - autoplay experiment setting being tested. 15 Flags - autoplay experiment setting being tested.
(...skipping 315 matching lines...) Expand 10 before | Expand all | Expand 10 after
329 queueNextExperiment(); 331 queueNextExperiment();
330 else 332 else
331 runOneConfig(spec); 333 runOneConfig(spec);
332 } 334 }
333 335
334 window.internals.settings.setMediaPlaybackRequiresUserGesture(true); 336 window.internals.settings.setMediaPlaybackRequiresUserGesture(true);
335 runNextConfig(); 337 runNextConfig();
336 338
337 </script> 339 </script>
338 </html> 340 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698