Support text track selection in video controls

third_party/WebKit/Source/core/css/mediaControlsNew.css

 /*
  * Copyright (C) 2009 Apple Inc.  All rights reserved.
  * Copyright (C) 2015 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 26 matching lines @@
 }
 
 audio::-webkit-media-controls,
 video::-webkit-media-controls {
     width: inherit;
     height: inherit;
     position: relative;
     direction: ltr;
     display: flex;
     flex-direction: column;
+    font-family: Segoe, "Helvetica Neue", Roboto, Arial, Helvetica, sans-serif ;

[philipj_slow, 2016/03/01 13:01:15]

While you're moving this, drop the extra space before the ;

[srivats, 2016/03/30 00:46:42]

Done.

     justify-content: flex-end;
     align-items: center;
 }
 
 audio::-webkit-media-controls-enclosure, video::-webkit-media-controls-enclosure {
     width: 100%;
     height: 48px;
     flex-shrink: 0;
     bottom: 0;
     text-indent: 0;
@@ skipping 16 matching lines @@
     z-index: 0;
     overflow: hidden;
     text-align: right;
     bottom: auto;
     height: 48px;
     min-width: 48px;
     background-color: #fafafa;
     /* The duration is also specified in MediaControlElements.cpp and LayoutTests/media/media-controls.js */
     transition: opacity 0.3s;
 
-    font-family: Segoe, "Helvetica Neue", Roboto, Arial, Helvetica, sans-serif ;
     font-size: 14px;
     font-weight: normal;  /* Make sure that we don't inherit non-defaults. */
     font-style: normal;
 }
 
 video:-webkit-full-page-media {
     margin: auto;
     position: absolute;
     top: 0;
     right: 0;
@@ skipping 267 matching lines @@
 }
 
 audio::-webkit-media-controls-fullscreen-volume-min-button, video::-webkit-media-controls-fullscreen-volume-min-button {
     display: none;
 }
 
 audio::-webkit-media-controls-fullscreen-volume-max-button, video::-webkit-media-controls-fullscreen-volume-max-button {
     display: none;
 }
 
+video::-internal-media-controls-text-track-list {
+    position: absolute;

[philipj_slow, 2016/03/01 13:01:15]

https://bugs.chromium.org/p/chromium/issues/detail?id=566499 is a bug report
about video controls overlapping things on the page, and the cause appears to be
positioned (position: relative or absolute) elements that change the paint
order. If you know of a simple way to avoid adding new cases of this that would
be good, but if it seems like it'd take a lot of changes to this CL, feel free
to leave it, it's not making matters worse right now.

[srivats, 2016/03/30 00:46:42]

Didn't get a chance to look at this.

+    bottom: 48px;
+    right: 5px;
+    background-color: #fafafa;
+    max-width: 50%;

[philipj_slow, 2016/03/01 13:01:16]

This doesn't seem to be achieving the intended effect, because the width is
limited to 150px even when the video is much wider than 300px and the text
doesn't fit.

[srivats, 2016/03/30 00:46:42]

I am retaining the max-width attribute and I changed min-width to be 150px
instead.

+    max-height: 250px;
+    width: 150px;
+    overflow-x: hidden;
+    overflow-y: auto;
+    text-overflow: ellipsis;

[philipj_slow, 2016/03/01 13:01:15]

Oh look, you're already using text-overflow:ellipsis. But it doesn't seem to
actually work, right? Should be on the item instead, perhaps?

[srivats, 2016/03/30 00:46:42]

Moved it to item and no it still doesn't do ellipsis. :(
The text just gets clipped at the moment.

+    margin-bottom: 5px;
+    white-space: nowrap;
+    font-size: 14px;
+    padding: 8px 0px;
+}
+
+video::-internal-media-controls-text-track-list-item {
+    display: block;
+    color: #424242;
+    text-align: start;
+    line-height: 40px;
+    padding-right: 16px;
+}
+
+video::-internal-media-controls-text-track-list-item:hover {
+    background-color: #e0e0e0;
+}
+
+video::-internal-media-controls-text-track-list-item-input {
+    -webkit-appearance: -internal-media-track-selection-checkmark;
+    position: relative;
+    visibility: hidden;
+    top: -2px;

[philipj_slow, 2016/03/01 13:01:15]

Discussed first in
https://codereview.chromium.org/1082533002/diff/80001/Source/core/css/mediaCo...

Do you mean that if you use a PNG that's just a single color, it will still look
misaligned compared to the text? Does this mean that vertical-align isn't
working as hoped, or why is that?

Fixing this would avoid the position:relative, which would be nice.

[srivats, 2016/03/30 00:46:42]

Removed

+    left: 0;
+    vertical-align: middle;
+    margin: 0 5px 0 0;
+    width: 16px;
+    height: 16px;
+    margin-left: 12px;
+}
+
+video::-internal-media-controls-text-track-list-item-input:checked {
+    visibility: visible;
+}
+
+video::-internal-media-controls-text-track-list-kind-captions {
+    -webkit-appearance: -internal-media-closed-captions-icon;
+    height: 20px;
+    width: 20px;
+    margin-left: 10px;
+    vertical-align: middle;
+}
+
+video::-internal-media-controls-text-track-list-kind-subtitles {
+    -webkit-appearance: -internal-media-subtitles-icon;
+    height: 20px;
+    width: 20px;
+    margin-left: 10px;
+    vertical-align: middle;
+}
+
 video::-webkit-media-text-track-container {
     position: relative;
     width: inherit;
     height: inherit;
     overflow: hidden;
 
     font: 22px sans-serif;
     text-align: center;
     color: rgba(255, 255, 255, 1);
 
@@ skipping 54 matching lines @@
     font-weight: bold;
 }
 
 video::cue(u) {
     text-decoration: underline;
 }
 
 video::cue(i) {
     font-style: italic;
 }