Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Unified Diff: third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html

Issue 2334803002: Implementation of media controls download button. (Closed)
Patch Set: fixed tests Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html
diff --git a/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html b/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html
index 430b5672c88ab7a5df7a02844b09dac408348170..48f6090a0e04a211dd509d26ea4679d24e3b81b1 100644
--- a/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html
+++ b/third_party/WebKit/LayoutTests/media/video-controls-overflow-menu-text.html
@@ -11,6 +11,8 @@
<video controls></video>
<script>
async_test(function(t) {
+ if (window.internals)
+ window.internals.runtimeFlags.mediaControlsDownloadButtonEnabled = true;
// Set up video
var video = document.querySelector("video");
video.src = findMediaFile("video", "content/test");
@@ -20,7 +22,6 @@ async_test(function(t) {
internals.mediaPlayerRemoteRouteAvailabilityChanged(video, true);
video.onloadeddata = t.step_func_done(function() {
- window.internals.runtimeFlags.MediaControlsDownloadButton = true;
var overflowList = getOverflowList(video);
var children = overflowList.children;
// Ensure that all of the buttons are visible in the right order

Powered by Google App Engine
This is Rietveld 408576698