Media Controls: Use native control in ChromeOS video player |
||
Issue descriptionUse native controls in ChromeOS video player
,
Dec 15
,
Jan 17
(5 days ago)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ef3d7d28a446c4ca7c13dbbe26cde907391d45f2 commit ef3d7d28a446c4ca7c13dbbe26cde907391d45f2 Author: Jazz Xu <jazzhsu@chromium.org> Date: Thu Jan 17 19:37:17 2019 [Media Controls] toggle native controls with separate logic Completely separate current controls and native controls. Only have basic control functions in native controls. (implemented previous video and next video in this CL as well) Will continue adding back features in follow up CLs Bug: 909963 Change-Id: If14ea5a33b07070f16c760763b2770a952012180 Reviewed-on: https://chromium-review.googlesource.com/c/1383148 Commit-Queue: Jazz Xu <jazzhsu@chromium.org> Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#623799} [modify] https://crrev.com/ef3d7d28a446c4ca7c13dbbe26cde907391d45f2/ui/file_manager/video_player/js/BUILD.gn [modify] https://crrev.com/ef3d7d28a446c4ca7c13dbbe26cde907391d45f2/ui/file_manager/video_player/js/video_player.js [add] https://crrev.com/ef3d7d28a446c4ca7c13dbbe26cde907391d45f2/ui/file_manager/video_player/js/video_player_native_controls.js [modify] https://crrev.com/ef3d7d28a446c4ca7c13dbbe26cde907391d45f2/ui/file_manager/video_player/js/video_player_scripts.js |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Dec 14