Adding overflow menu to media player

third_party/WebKit/Source/core/css/CSSValueKeywords.in

Index: third_party/WebKit/Source/core/css/CSSValueKeywords.in
diff --git a/third_party/WebKit/Source/core/css/CSSValueKeywords.in b/third_party/WebKit/Source/core/css/CSSValueKeywords.in
index bea9811bf311804ceb9e55d335b6d0610ba2756f..0b6ec8e94b6eefb2cea01c12a51cf198cd8ac442 100644
--- a/third_party/WebKit/Source/core/css/CSSValueKeywords.in
+++ b/third_party/WebKit/Source/core/css/CSSValueKeywords.in
@@ -647,6 +647,7 @@ media-time-remaining-display
 -internal-media-track-selection-checkmark
 -internal-media-closed-captions-icon
 -internal-media-subtitles-icon
+media-overflow-menu-button

[whywhat, 2016/08/19 23:56:55]

why are there -internal and public keywords? is it related to whether the page
can access the element using css? if so, we probably want the overflow button to
be internal?

[kdsilva, 2016/08/24 05:33:25]

It's not obvious to me what the naming convention is for this file. For the
media controls, I noticed that the all of the buttons in the media player were
public (mute button, play button, closed captions button, etc) so that was my
reasoning for making the overflow button public. 

[Timothy Loh, 2016/08/24 06:19:38]

We can't just go and add whatever we want as web-exposed features. Making this
-internal means that web devs can't set this value themselves.

[kdsilva, 2016/08/26 18:11:02]

Acknowledged.

 menulist
 menulist-button
 menulist-text