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

Unified Diff: LayoutTests/media/media-captions-no-controls.html

Issue 411833006: Use the mediaControlsButton() helper more in media tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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
« no previous file with comments | « no previous file | LayoutTests/media/media-captions-no-controls-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/media/media-captions-no-controls.html
diff --git a/LayoutTests/media/media-captions-no-controls.html b/LayoutTests/media/media-captions-no-controls.html
index 32c6faeb8701b8aa018b00333363e8f248433db6..e72227929e77415a6af12ced586716b32f12203f 100644
--- a/LayoutTests/media/media-captions-no-controls.html
+++ b/LayoutTests/media/media-captions-no-controls.html
@@ -25,9 +25,9 @@
run("video.setAttribute('controls','controls')");
testExpected("video.textTracks.length", 1);
testExpected("video.controls", true);
- controlsElement = mediaControlsElement(internals.shadowRoot(video).firstChild, "-webkit-media-controls-panel");
+ panel = mediaControlsButton(video, "panel");
testExpected("internals.shadowRoot(video).firstChild", null, "!=");
- testExpected("controlsElement.style['display']", 'none', "!=");
+ testExpected("panel.style['display']", 'none', "!=");
consoleWrite("");
endTest();
« no previous file with comments | « no previous file | LayoutTests/media/media-captions-no-controls-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698