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

Unified Diff: chrome/test/data/webui/media_router/media_router_header_tests.js

Issue 1963933002: [Media Router WebUI] Run afterNextRender() before tests using media-router-container. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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: chrome/test/data/webui/media_router/media_router_header_tests.js
diff --git a/chrome/test/data/webui/media_router/media_router_header_tests.js b/chrome/test/data/webui/media_router/media_router_header_tests.js
index 0d3292bd3a9bfb0bc3f3010bc9066b7f061daf45..841fe457148337a71f63d2c900f0d1a447835289 100644
--- a/chrome/test/data/webui/media_router/media_router_header_tests.js
+++ b/chrome/test/data/webui/media_router/media_router_header_tests.js
@@ -7,6 +7,12 @@ cr.define('media_router_header', function() {
function registerTests() {
suite('MediaRouterHeader', function() {
/**
+ * Media Router Container created before each test.
+ * @type {?MediaRouterContainer}
+ */
+ var container;
+
+ /**
* Media Router Header created before each test.
* @type {MediaRouterHeader}
*/
@@ -72,8 +78,9 @@ cr.define('media_router_header', function() {
document.body.appendChild(container);
header = container.$['container-header'];
- // Allow for the media router header to be created and attached.
- setTimeout(done);
+ // Allow for the media router container to be created, attached, and
+ // listeners registered in an afterNextRender() call.
+ Polymer.RenderStatus.afterNextRender(this, done);
});
// Tests for 'close-dialog' event firing when the close button is

Powered by Google App Engine
This is Rietveld 408576698