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

Side by Side Diff: ManualTests/media-players-are-dropped-on-error.html

Issue 11275156: Merge 133252 - HTMLMediaPlayer should free m_player when src is set/changed (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1312/
Patch Set: Created 8 years, 1 month 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 | « ChangeLog ('k') | Source/WebCore/ChangeLog » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script> 3 <script>
4 var urls = [ 4 var urls = [
5 "file:///does not exist oh noes/test.mp4", 5 "file:///does not exist oh noes/test.mp4",
6 "../LayoutTests/media/content/test-25fps.mp4" 6 "../LayoutTests/media/content/test-25fps.mp4"
7 ]; 7 ];
8 var kickoffFunctions = [ 8 var kickoffFunctions = [
9 "load", 9 "load",
10 "play" 10 "play"
(...skipping 20 matching lines...) Expand all
31 } 31 }
32 </script> 32 </script>
33 </head> 33 </head>
34 <body onload="setInterval('releaseAndAddMediaElements()', 100)"> 34 <body onload="setInterval('releaseAndAddMediaElements()', 100)">
35 Test that media players aren't leaked on error. 35 Test that media players aren't leaked on error.
36 Load this page and verify the number of threads used by the browser doesn't 36 Load this page and verify the number of threads used by the browser doesn't
37 seem unreasonable (e.g. chrome uses 4-5 threads per video tag so staying 37 seem unreasonable (e.g. chrome uses 4-5 threads per video tag so staying
38 under 100 threads is "success", since this instantiates 20 &lt;video&gt; ele ments). 38 under 100 threads is "success", since this instantiates 20 &lt;video&gt; ele ments).
39 </body> 39 </body>
40 </html> 40 </html>
OLDNEW
« no previous file with comments | « ChangeLog ('k') | Source/WebCore/ChangeLog » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698