From cbf0be7f1e4b3b9b492a3ff7f06f0d5bfae37029 Mon Sep 17 00:00:00 2001
From: Anton Vayvod <avayvod@google.com>
Date: Thu, 9 Mar 2017 06:42:36 -0500
Subject: [PATCH 1/2] [HTMLMediaElement] Add a controlsList/controlslist
 attribute.

Adds a controlslist content attribute reflected by a DOMTokenList dom
attribute named controlsList to the HTMLMediaElement and <audio>/<video>
tags.

Specifies three supported keywords: nodownload, nofullscreen, and
noremoteplayback.

Documents the change proposed in https://github.com/whatwg/html/issues/2293
---
 source | 55 ++++++++++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 52 insertions(+), 3 deletions(-)

diff --git a/source b/source
index f61387a437b..aa22b031feb 100644
--- a/source
+++ b/source
@@ -30194,6 +30194,7 @@ interface <dfn>HTMLParamElement</dfn> : <span>HTMLElement</span> {
    <dd><code data-x="attr-media-loop">loop</code></dd>
    <dd><code data-x="attr-media-muted">muted</code></dd>
    <dd><code data-x="attr-media-controls">controls</code></dd>
+   <dd><code data-x="attr-media-controlslist">controlslist</code></dd>
    <dd><code data-x="attr-dim-width">width</code></dd>
    <dd><code data-x="attr-dim-height">height</code></dd>
    <dt><span data-x="concept-element-dom">DOM interface</span>:</dt><!--TOPIC:DOM APIs-->
@@ -30586,6 +30587,7 @@ zero or more <code>track</code> elements, then
    <dd><code data-x="attr-media-loop">loop</code></dd>
    <dd><code data-x="attr-media-muted">muted</code></dd>
    <dd><code data-x="attr-media-controls">controls</code></dd>
+   <dd><code data-x="attr-media-controlslist">controlslist</code></dd>
    <dt><span data-x="concept-element-dom">DOM interface</span>:</dt><!--TOPIC:DOM APIs-->
    <dd>
     <pre class="idl">[<span>HTMLConstructor</span>, NamedConstructor=<span data-x="dom-Audio">Audio</span>(optional DOMString src)]
@@ -31033,6 +31035,9 @@ interface <dfn>HTMLMediaElement</dfn> : <span>HTMLElement</span> {
 
   // controls
   [<span>CEReactions</span>] attribute boolean <span data-x="dom-media-controls">controls</span>;
+  [<span>CEReactions</span>, SameObject, PutForwards=<span
+      data-x="dom-DOMTokenList-value">value</span>] readonly attribute <span>DOMTokenList</span>
+      <span data-x="dom-media-controlslist">controlsList</span>;
   attribute double <span data-x="dom-media-volume">volume</span>;
   attribute boolean <span data-x="dom-media-muted">muted</span>;
   [<span>CEReactions</span>] attribute boolean <span data-x="dom-media-defaultMuted">defaultMuted</span>;
@@ -31047,8 +31052,9 @@ interface <dfn>HTMLMediaElement</dfn> : <span>HTMLElement</span> {
   <p>The <dfn>media element attributes</dfn>, <code data-x="attr-media-src">src</code>, <code
   data-x="attr-media-crossorigin">crossorigin</code>, <code
   data-x="attr-media-preload">preload</code>, <code data-x="attr-media-autoplay">autoplay</code>,
-  <code data-x="attr-media-loop">loop</code>, <code data-x="attr-media-muted">muted</code>, and
-  <code data-x="attr-media-controls">controls</code>, apply to all <span data-x="media
+  <code data-x="attr-media-loop">loop</code>, <code data-x="attr-media-muted">muted</code>, <code
+  data-x="attr-media-controls">controls</code>, and <code
+  data-x="attr-media-controlslist">controlslist</code> apply to all <span data-x="media
   element">media elements</span>. They are defined in this section.</p>
 
   <!-- proposed v2 (actually v3!) features:
@@ -36503,6 +36509,41 @@ red:89
   <p>The <dfn><code data-x="dom-media-controls">controls</code></dfn> IDL attribute must
   <span>reflect</span> the content attribute of the same name.</p>
 
+  </div>
+
+  <p>The <dfn><code data-x="attr-media-controlslist">controlslist</code></dfn> attribute, when
+  specified, helps the user agent select what controls to show on the media element whenever
+  the user agent shows its own set of controls. Its value must be an <span>unordered set of unique
+  space-separated tokens</span> that are <span>ASCII case-insensitive</span>. The allowed values
+  are <dfn><code data-x="attr-media-controlslist-nodownload">nodownload</code></dfn>, <dfn><code
+  data-x="attr-media-controlslist-nofullscreen">nofullscreen</code></dfn>, and
+  <dfn><code data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code></dfn>.
+
+  <p>The <code data-x="attr-media-controlslist-nodownload">nodownload</code> keyword causes the
+  download control to be hidden when using the user agent's own set of controls for the media
+  element. The <code data-x="attr-media-controlslist-nofullscreen">nofullscreen</code> keyword
+  causes the fullscreen mode control to be hidden when using the user agent's own set of controls
+  for the media element. The <code
+  data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code> keyword causes the remote
+  playback control to be hidden when using the user agent's own set of controls for the media
+  element.</p>
+
+  <div w-nodev>
+
+  <p>The <dfn><code data-x="dom-media-controlslist">controlsList</code></dfn> IDL attribute must
+  <span>reflect</span> the value of the <span data-x="attr-media-controlslist">controlslist</span>
+  content attribute.</p>
+
+  <p>The <span data-x="concept-supported-tokens">supported tokens</span> for <code
+  data-x="dom-media-controlslist">controlsList</code>'s <code>DOMTokenList</code> are the allowed
+  values defined in the <code data-x="attr-media-controlslist">controlslist</code> attribute and
+  supported by the user agent.</p>
+
+  <p>The user agent might ignore the author's preference depending on the environment, e.g.
+  <code data-x="attr-media-controlslist-nofullscreen">nofullscreen</code> keyword doesn't mean the
+  fullscreen control MUST be hidden for the user if the content area containing the video is small
+  (e.g. on a mobile device).</p>
+
   <hr>
 
   </div>
@@ -114158,7 +114199,8 @@ interface <dfn>External</dfn> {
          <code data-x="attr-media-autoplay">autoplay</code>;
          <code data-x="attr-media-loop">loop</code>;
          <code data-x="attr-media-muted">muted</code>;
-         <code data-x="attr-media-controls">controls</code></td>
+         <code data-x="attr-media-controls">controls</code>
+         <code data-x="attr-media-controlslist">controlslist</code></td>
      <td><code>HTMLAudioElement</code></td>
     </tr>
 
@@ -115652,6 +115694,7 @@ interface <dfn>External</dfn> {
          <code data-x="attr-media-loop">loop</code>;
          <code data-x="attr-media-muted">muted</code>;
          <code data-x="attr-media-controls">controls</code>;
+         <code data-x="attr-media-controlslist">controlslist</code>;
          <code data-x="attr-dim-width">width</code>;
          <code data-x="attr-dim-height">height</code></td>
      <td><code>HTMLVideoElement</code></td>
@@ -116267,6 +116310,12 @@ interface <dfn>External</dfn> {
           <code data-x="attr-media-controls">video</code>
      <td> Show user agent controls
      <td> <span>Boolean attribute</span>
+    <tr>
+     <th> <code data-x="">controlsList</code>
+     <td> <code data-x="attr-media-controlslist">audio</code>;
+          <code data-x="attr-media-controlslist">video</code>
+     <td> Show/hide specific user agent controls
+     <td> <span>DOMTokenList</span> attribute
     <tr>
      <th> <code data-x="">coords</code>
      <td> <code data-x="attr-area-coords">area</code>

From 2a4fc93dd3a922073a5f6b683d561bf3f81e4120 Mon Sep 17 00:00:00 2001
From: Anton Vayvod <avayvod@google.com>
Date: Thu, 9 Mar 2017 15:32:37 -0500
Subject: [PATCH 2/2] Addressed Domenic's comments

---
 source | 40 ++++++++++++++++++++++++----------------
 1 file changed, 24 insertions(+), 16 deletions(-)

diff --git a/source b/source
index aa22b031feb..44ae9d936c4 100644
--- a/source
+++ b/source
@@ -31035,9 +31035,8 @@ interface <dfn>HTMLMediaElement</dfn> : <span>HTMLElement</span> {
 
   // controls
   [<span>CEReactions</span>] attribute boolean <span data-x="dom-media-controls">controls</span>;
-  [<span>CEReactions</span>, SameObject, PutForwards=<span
-      data-x="dom-DOMTokenList-value">value</span>] readonly attribute <span>DOMTokenList</span>
-      <span data-x="dom-media-controlslist">controlsList</span>;
+  [SameObject, PutForwards=<span data-x="dom-DOMTokenList-value">value</span>] readonly attribute
+      <span>DOMTokenList</span> <span data-x="dom-media-controlslist">controlsList</span>;
   attribute double <span data-x="dom-media-volume">volume</span>;
   attribute boolean <span data-x="dom-media-muted">muted</span>;
   [<span>CEReactions</span>] attribute boolean <span data-x="dom-media-defaultMuted">defaultMuted</span>;
@@ -36519,14 +36518,18 @@ red:89
   data-x="attr-media-controlslist-nofullscreen">nofullscreen</code></dfn>, and
   <dfn><code data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code></dfn>.
 
-  <p>The <code data-x="attr-media-controlslist-nodownload">nodownload</code> keyword causes the
-  download control to be hidden when using the user agent's own set of controls for the media
+  <p>The <code data-x="attr-media-controlslist-nodownload">nodownload</code> keyword hints that the
+  download control should be hidden when using the user agent's own set of controls for the media
   element. The <code data-x="attr-media-controlslist-nofullscreen">nofullscreen</code> keyword
-  causes the fullscreen mode control to be hidden when using the user agent's own set of controls
-  for the media element. The <code
-  data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code> keyword causes the remote
-  playback control to be hidden when using the user agent's own set of controls for the media
-  element.</p>
+  hints that the fullscreen mode control should be hidden when using the user agent's own set of
+  controls for the media element. The <code
+  data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code> keyword hints that the
+  remote playback control should be hidden when using the user agent's own set of controls for the
+  media element.</p>
+
+  <p class="note">Hiding these aspects of the user agent's own controls does not necessarily disable
+  the related functionality. For example, the user agent might present the same functionality
+  through a context menu or keyboard shortcut.</p>
 
   <div w-nodev>
 
@@ -36539,10 +36542,11 @@ red:89
   values defined in the <code data-x="attr-media-controlslist">controlslist</code> attribute and
   supported by the user agent.</p>
 
-  <p>The user agent might ignore the author's preference depending on the environment, e.g.
-  <code data-x="attr-media-controlslist-nofullscreen">nofullscreen</code> keyword doesn't mean the
-  fullscreen control MUST be hidden for the user if the content area containing the video is small
-  (e.g. on a mobile device).</p>
+  <p>A user agent MAY ignore the author's preference when it makes sense.</p>
+
+  <p class="example">A user agent might ignore the <code
+  data-x="attr-media-controlslist-nofullscreen">nofullscreen</code> keyword if the content area
+  containing the video is small, such as on a mobile device.</p>
 
   <hr>
 
@@ -116311,11 +116315,15 @@ interface <dfn>External</dfn> {
      <td> Show user agent controls
      <td> <span>Boolean attribute</span>
     <tr>
-     <th> <code data-x="">controlsList</code>
+     <th> <code data-x="">controlslist</code>
      <td> <code data-x="attr-media-controlslist">audio</code>;
           <code data-x="attr-media-controlslist">video</code>
      <td> Show/hide specific user agent controls
-     <td> <span>DOMTokenList</span> attribute
+     <td> <span>Unordered set of unique space-separated tokens</span>, <span>ASCII
+     case-insensitive</span>, consisting of "<code
+     data-x="attr-media-controlslist-nodownload">nodownload</code>", "<code
+     data-x="attr-media-controlslist-nofullscreen">nofullscreen</code>", and "<code
+     data-x="attr-media-controlslist-noremoteplayback">noremoteplayback</code>".
     <tr>
      <th> <code data-x="">coords</code>
      <td> <code data-x="attr-area-coords">area</code>