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

Side by Side Diff: third_party/WebKit/LayoutTests/media/overflow-menu.js

Issue 2243473002: Adding overflow menu to media player (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 3 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
OLDNEW
(Empty)
1 // We expect the items in the overflow to appear in the following ordering.
2 var overflowButtonsCSS = ["-webkit-media-controls-mute-button",
mlamouri (slow - plz ping) 2016/09/06 10:25:44 nit: for readability, I would break the line after
kdsilva 2016/09/06 17:32:45 Done.
3 "-internal-media-controls-cast-button",
4 "-webkit-media-controls-toggle-closed-captions-button",
5 "-webkit-media-controls-fullscreen-button",
6 "-webkit-media-controls-play-button"];
7 // PseudoID for the overflow button
8 var menuID = "-internal-overflow-menu-button";
9 // PseudoID for the overflow list
10 var listID = "-internal-media-controls-overflow-menu-list";
11
12 // Returns the overflow menu button within the given media element
13 function getOverflowMenuButton(media) {
14 return mediaControlsElement(internals.shadowRoot(media).firstChild, menuID);
15 }
16
17 // Returns the overflow menu list of overflow controls
18 function getOverflowList(media) {
19 return mediaControlsElement(internals.shadowRoot(media).firstChild, listID);
20 }
21
22 // Location of media control element in the overflow button
23 var OverflowMenuButtons = {
24 MUTE: 0,
25 CAST: 1,
26 CLOSED_CAPTIONS: 2,
27 FULLSCREEN: 3,
28 PLAY: 4,
29 };
30
31 // Default text within the overflow menu
32 var overflowMenuText = ["Mute", "Cast", "Captions", "Fullscreen", "Play"];
33
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698