third_party/WebKit/LayoutTests/media/controls-repaint-for-network-change-expected.html - Issue 1417683004: Media controls refer to less magic state.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/media/controls-repaint-for-network-change-expected.html

Issue 1417683004: Media controls refer to less magic state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: rebased, fixed assert. Created 5 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:

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <style>

	3 /* restrict to just the play button, so the slider doesn't mismatch */

	4 audio {

	5 width: 40px;

	6 height: 40px;

	7 }

	8 </style>

	9 <script src="../resources/run-after-layout-and-paint.js"></script>

	10 <script src=media-file.js></script>

	11 <script>

	12

	13 testRunner.waitUntilDone();

	14

	15 function test() {

	16 var src = findMediaFile("audio", "content/empty");

	17 var audios = document.getElementById("audio1").src = src;
	philipj_slow 2015/10/30 10:55:36 The audios variable is unused. The audios variable is unused. liberato (no reviews please) 2015/10/30 15:09:07 Done. Show quoted text On 2015/10/30 10:55:36, philipj wrote: > The audios variable is unused. Done.
	18

	19 runAfterLayoutAndPaint(function() {

	20 testRunner.notifyDone();

	21 });

	22 }

	23 </script>

	24

	25 <body onload="test()">

	26 <audio id="audio0" controls></audio>

	27 <audio id="audio1" controls></audio>

	28 <audio id="audio2" controls></audio>

	29 <p>Test that network state transitions paint audio controls properly.</p>

	30 </body>

	31

OLD	NEW