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

Side by Side Diff: LayoutTests/media/video-intrinsic-width-height.html

Issue 38313002: Remove <video> width and height attributes from intrisic size logic (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: learn to indent Created 7 years, 2 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
« no previous file with comments | « no previous file | LayoutTests/media/video-intrinsic-width-height-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!DOCTYPE html>
2 <html>
3 <head>
4 <title>&lt;video> intrinsic width/height</title>
5 <script src="../resources/testharness.js"></script>
6 <script src="../resources/testharnessreport.js"></script>
7 </head>
8 <body>
9 <div id="log"></div>
10 <!-- These tests assume that the default intrinsic width is 300x150, so that the default
11 intrinsic ratio is 2:1. This is no longer per spec, but is what is implemented. -->
12 <video title="no width/height attributes"
13 data-expected-width="300" data-expected-height="150"></video>
14 <video title="only width attribute"
15 data-expected-width="100" data-expected-height="50"
16 width="100"></video>
17 <video title="only height attribute"
18 data-expected-width="200" data-expected-height="100"
19 height="100"></video>
20 <video title="both width/height attributes"
21 data-expected-width="100" data-expected-height="100"
22 width="100" height="100"></video>
23 <!-- A width:height ratio other than 2:1 and overriding the specified st yle must be used to
24 verify that width/height does not influence intrinsic ratio -->
25 <video title="both width/height attributes and style"
26 data-expected-width="300" data-expected-height="150"
27 width="100" height="100" style="width: auto; height: auto"></vide o>
28 <script>
29 Array.prototype.forEach.call(document.querySelectorAll('video'), fun ction(video)
30 {
31 test(function()
32 {
33 assert_equals(video.clientWidth, parseInt(video.dataset.expe ctedWidth), "width");
34 assert_equals(video.clientHeight, parseInt(video.dataset.exp ectedHeight), "height");
35 }, video.title);
36 });
37 </script>
38 </body>
39 </html>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/media/video-intrinsic-width-height-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698