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

Side by Side Diff: third_party/WebKit/Source/web/tests/data/link-manifest-change.html

Issue 1793933002: Don't trigger a link resource fetch on media change. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed test and made mediaChanged() virtual 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 <!-- This file is meant to test how many times the WebFrameClient will be 2 <!-- This file is meant to test how many times the WebFrameClient will be
3 notified of a change in the current manifest URL associated with the document. --> 3 notified of a change in the current manifest URL associated with the document. -->
4 <html> 4 <html>
5 <head> 5 <head>
6 <link rel='manifest' href=''> 6 <link rel='manifest' href=''>
7 <link rel='manifest'> 7 <link rel='manifest'>
8 <link rel='manifest foo'> 8 <link rel='manifest foo'>
9 <link rel='manifest' crossorigin foo bar> 9 <link rel='manifest' crossorigin foo bar>
10 <!-- 4 times (= 4) --> 10 <!-- 4 times (= 4) -->
(...skipping 14 matching lines...) Expand all
25 25
26 document.head.appendChild(createLink('foo.json', 'manifest')); 26 document.head.appendChild(createLink('foo.json', 'manifest'));
27 document.head.appendChild(createLink('foo.json', '')); 27 document.head.appendChild(createLink('foo.json', ''));
28 document.head.appendChild(createLink('foo.json', 'manifest')); 28 document.head.appendChild(createLink('foo.json', 'manifest'));
29 document.head.insertBefore(createLink('foo.json', 'manifest'), links[0]); 29 document.head.insertBefore(createLink('foo.json', 'manifest'), links[0]);
30 // 3 times (= 8) 30 // 3 times (= 8)
31 31
32 links[0].crossOrigin = 'use-credentials'; 32 links[0].crossOrigin = 'use-credentials';
33 links[0].media = ''; 33 links[0].media = '';
34 links[0].type = 'image/gif'; 34 links[0].type = 'image/gif';
35 // 2 times (= 10) 35 // 1 times (= 9)
kochi 2016/03/14 05:59:34 s/times/time/ ?
36 36
37 links[0].href = 'bar.json'; 37 links[0].href = 'bar.json';
38 links[0].rel = 'blah'; 38 links[0].rel = 'blah';
39 links[0].rel = 'manifest'; 39 links[0].rel = 'manifest';
40 // 3 times (= 13) 40 // 3 times (= 12)
41 41
42 document.head.removeChild(links[0]); 42 document.head.removeChild(links[0]);
43 // 1 time (= 14) 43 // 1 time (= 13)
44 </script> 44 </script>
45 </html> 45 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698