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

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/media/preload-conditions.html

Issue 2546993003: [MediaElement] Allow preloading for non-network scheme src (Closed)
Patch Set: nits Created 4 years 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/media/preload-conditions.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <title>Test media preloading behaviour with different conditions.</title> 2 <title>Test media preloading behaviour with different conditions.</title>
3 <script src="../resources/testharness.js"></script> 3 <script src="../resources/testharness.js"></script>
4 <script src="../resources/testharnessreport.js"></script> 4 <script src="../resources/testharnessreport.js"></script>
5 <script src="media-file.js"></script> 5 <script src="../../media-resources/media-file.js"></script>
6 <script> 6 <script>
7 var tests = [ 7 var tests = [
8 { 8 {
9 name: 'regular conditions', 9 name: 'regular conditions',
10 conditions: { 10 conditions: {
11 dataSaver: false, 11 dataSaver: false,
12 cellular: false, 12 cellular: false,
13 lowEndDevice: false, 13 lowEndDevice: false,
14 }, 14 },
15 expectations: { 15 expectations: {
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 // Test not allowed values. 128 // Test not allowed values.
129 [ '', 'default', 'foo' ].forEach(preload => { 129 [ '', 'default', 'foo' ].forEach(preload => {
130 checkPreloadAttribute(media, preload, test.expectations.default); 130 checkPreloadAttribute(media, preload, test.expectations.default);
131 }); 131 });
132 132
133 // Test loading events. 133 // Test loading events.
134 var expectedLoading = 4; 134 var expectedLoading = 4;
135 [ '', 'none', 'metadata', 'auto' ].forEach(preload => { 135 [ '', 'none', 'metadata', 'auto' ].forEach(preload => {
136 var media = document.createElement('video'); 136 var media = document.createElement('video');
137 media.preload = preload; 137 media.preload = preload;
138 media.src = findMediaFile('video', 'content/test'); 138 media.src = findMediaFile('video', 'resources/test');
139 assert_equals(media.readyState, HTMLMediaElement.HAVE_NOTHING); 139 assert_equals(media.readyState, HTMLMediaElement.HAVE_NOTHING);
140 140
141 switch (media.preload) { 141 switch (media.preload) {
142 case 'none': 142 case 'none':
143 media.onloadedmetadata = t.unreached_func(); 143 media.onloadedmetadata = t.unreached_func();
144 media.onprogress = t.unreached_func(); 144 media.onprogress = t.unreached_func();
145 145
146 media.onsuspend = t.step_func(_ => { 146 media.onsuspend = t.step_func(_ => {
147 assert_equals(media.readyState, HTMLMediaElement.HAVE_NOTHING); 147 assert_equals(media.readyState, HTMLMediaElement.HAVE_NOTHING);
148 --expectedLoading; 148 --expectedLoading;
(...skipping 22 matching lines...) Expand all
171 runNextTest(); 171 runNextTest();
172 }); 172 });
173 break; 173 break;
174 } 174 }
175 }); 175 });
176 } 176 }
177 177
178 asyncTests[currentTest].step_func(runTest(asyncTests[currentTest], tests[current Test])); 178 asyncTests[currentTest].step_func(runTest(asyncTests[currentTest], tests[current Test]));
179 179
180 </script> 180 </script>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/media/preload-conditions.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698