Adds keyboard functionality for videos.

third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp

Index: third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp
diff --git a/third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp b/third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp
index 070f17b32971c62841984ede8a0b15d1d8086e8d..dc9474ba4a5c6bc2daaab34e2041d9d4abb131ea 100644
--- a/third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp
+++ b/third_party/WebKit/Source/core/html/shadow/MediaControlsMediaEventListener.cpp
@@ -29,6 +29,9 @@ void MediaControlsMediaEventListener::attach() {
   mediaElement().addEventListener(EventTypeNames::durationchange, this, false);
   mediaElement().addEventListener(EventTypeNames::error, this, false);
   mediaElement().addEventListener(EventTypeNames::loadedmetadata, this, false);
+  mediaElement().addEventListener(EventTypeNames::keypress, this, false);
+  mediaElement().addEventListener(EventTypeNames::keydown, this, false);
+  mediaElement().addEventListener(EventTypeNames::keyup, this, false);

[mlamouri (slow - plz ping), 2017/03/27 13:02:12]

I think keypress should be enough here.

[CJ, 2017/04/08 00:31:27]

Keypress only works for the Enter button. All other controls are not registered.
Need to check if keyup has any baring in the blue line error once the fix for
that makes its way into the repo.

 
   // Listen to two different fullscreen events in order to make sure the new and
   // old APIs are handled.
@@ -122,6 +125,12 @@ void MediaControlsMediaEventListener::handleEvent(
     return;
   }
 
+  if (event->type() == EventTypeNames::keypress || EventTypeNames::keydown ||
+      EventTypeNames::keyup) {

[mlamouri (slow - plz ping), 2017/03/27 13:02:12]

ditto

[CJ, 2017/04/08 00:31:27]

Acknowledged.

+    m_mediaControls->onMediaKeypress(event);
+    return;
+  }
+
   NOTREACHED();
 }