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

Side by Side Diff: third_party/WebKit/LayoutTests/media/controls-cast-button.html

Issue 2684973004: Remove unnecessary refreshCastButtonVisibility call (Closed)
Patch Set: remove extra blank lines Created 3 years, 10 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 | third_party/WebKit/LayoutTests/media/controls-cast-button-narrow.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <title>media controls cast button</title> 2 <title>media controls cast button</title>
3 <script src="../resources/testharness.js"></script> 3 <script src="../resources/testharness.js"></script>
4 <script src="../resources/testharnessreport.js"></script> 4 <script src="../resources/testharnessreport.js"></script>
5 <script src="media-file.js"></script> 5 <script src="media-file.js"></script>
6 <script src="media-controls.js"></script> 6 <script src="media-controls.js"></script>
7 <video controls width="500"></video> 7 <video controls width="500"></video>
8 <script> 8 <script>
9 async_test(function(t) { 9 async_test(function(t) {
10 var video = document.querySelector("video"); 10 var video = document.querySelector("video");
(...skipping 25 matching lines...) Expand all
36 assert_equals(overlayButton.style.display, "none", "Overlay button shoul d not be visible with controls"); 36 assert_equals(overlayButton.style.display, "none", "Overlay button shoul d not be visible with controls");
37 37
38 // And to the right of the fullscreen button 38 // And to the right of the fullscreen button
39 var fullscreenPosition = mediaControlsButtonCoordinates(video, "fullscre en-button"); 39 var fullscreenPosition = mediaControlsButtonCoordinates(video, "fullscre en-button");
40 assert_greater_than(x, fullscreenPosition[0], "button should be to right of fullscreen button"); 40 assert_greater_than(x, fullscreenPosition[0], "button should be to right of fullscreen button");
41 41
42 // Remove cast device - cast button should go away 42 // Remove cast device - cast button should go away
43 internals.mediaPlayerRemoteRouteAvailabilityChanged(video, false); 43 internals.mediaPlayerRemoteRouteAvailabilityChanged(video, false);
44 assert_equals(button.style.display, "none", "button should not be visibl e after cast device goes away"); 44 assert_equals(button.style.display, "none", "button should not be visibl e after cast device goes away");
45 }); 45 });
46
47 function castButton(element) {
48 var controlID = "-internal-media-controls-cast-button";
49 var button = mediaControlsElement(internals.shadowRoot(element).firstChi ld, controlID);
50 if (!button)
51 throw "Failed to find cast button";
52 return button;
53 }
54
55 function overlayCastButton(element) {
56 var controlID = "-internal-media-controls-overlay-cast-button";
57 var button = mediaControlsElement(internals.shadowRoot(element).firstChi ld, controlID);
58 if (!button)
59 throw "Failed to find cast button";
60 return button;
61 }
62 }); 46 });
63 </script> 47 </script>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/media/controls-cast-button-narrow.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698