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

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

Issue 2725503002: [Media Router] Custom Controls 4 - Implement details view WebUI (Closed)
Patch Set: Address Mark's comments Created 3 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/route_details_tests.js
diff --git a/chrome/test/data/webui/media_router/route_details_tests.js b/chrome/test/data/webui/media_router/route_details_tests.js
index dcab054364afd556eb40b7317709307441ca9cc8..8c7dcf0cfbf8a0a44149b01cda8e48d74d6ffcc0 100644
--- a/chrome/test/data/webui/media_router/route_details_tests.js
+++ b/chrome/test/data/webui/media_router/route_details_tests.js
@@ -24,6 +24,12 @@ cr.define('route_details', function() {
*/
var fakeRouteTwo;
+ /**
+ * Fake sink that corresponds to |fakeRouteOne|.
+ * @type {media_router.Sink}
+ */
+ var fakeSinkOne;
+
// Checks whether |expected| and the text in the span element in
// the |elementId| element are equal.
var checkSpanText = function(expected, elementId) {
@@ -37,13 +43,13 @@ cr.define('route_details', function() {
assertTrue(details.$['custom-controller'].hasAttribute('hidden'));
};
- // Checks the default route view is shown.
+ // Checks the start button is shown.
var checkStartCastButtonIsShown = function() {
assertFalse(
details.$['start-casting-to-route-button'].hasAttribute('hidden'));
};
- // Checks the default route view is not shown.
+ // Checks the start button is not shown.
var checkStartCastButtonIsNotShown = function() {
assertTrue(
details.$['start-casting-to-route-button'].hasAttribute('hidden'));
@@ -61,12 +67,6 @@ cr.define('route_details', function() {
assertEquals(expected, details.$[elementId].innerText);
};
- /**
- * Fake sink that corresponds to |fakeRouteOne|.
- * @type {media_router.Sink}
- */
- var fakeSinkOne;
-
// Import route_details.html before running suite.
suiteSetup(function() {
return PolymerTest.importHtml(

Powered by Google App Engine
This is Rietveld 408576698