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

Unified Diff: LayoutTests/http/tests/media/video-buffering-repaints-controls.html

Issue 8496044: Repaint video controls when buffering during pause. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Nits, drop whitespace. Created 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/WebCore/html/HTMLMediaElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/http/tests/media/video-buffering-repaints-controls.html
diff --git a/LayoutTests/http/tests/media/video-buffering-repaints-controls.html b/LayoutTests/http/tests/media/video-buffering-repaints-controls.html
new file mode 100644
index 0000000000000000000000000000000000000000..67fa31e93513ab5f24aca8c1e80a2150533c6345
--- /dev/null
+++ b/LayoutTests/http/tests/media/video-buffering-repaints-controls.html
@@ -0,0 +1,36 @@
+<html>
+ <head>
+ <script src=../../media-resources/media-file.js></script>
+ <script>
+ var painted = false;
+ function start() {
+ if (window.layoutTestController)
+ layoutTestController.waitUntilDone();
+
+ var video = document.getElementsByTagName('video')[0];
+
+ video.addEventListener('progress', function(event) {
+ // Each progress event should correlate to a repaint. Check
+ // by watching for a repaint between progress events.
+ if (painted) {
+ layoutTestController.notifyDone();
+ } else {
+ layoutTestController.display();
+ painted = true;
+ }
+ }, false);
+
+ var mediaFile = findMediaFile("video", "resources/test");
+ var mimeType = mimeTypeForFile(mediaFile);
+
+ // Don't actually play since we're testing via an image diff.
+ video.src = 'http://127.0.0.1:8000/media/video-throttled-load.cgi?&name=' + mediaFile + '&throttle=10&type=' + mimeType
+ }
+ </script>
+ </head>
+
+ <body onload="start();">
+ <p>Test that media controls repaint correctly during paused states when new data is buffered.</p>
+ <video controls preload></video><br/>
+ </body>
+</html>
« no previous file with comments | « no previous file | Source/WebCore/html/HTMLMediaElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698