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

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

Issue 2365743002: [MR UI] Add a wrapper div to the sink list and search results in the MR dialog (Closed)
Patch Set: Rebase, fix browser test Created 4 years, 2 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 | « chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/media_router/media_router_container_filter_tests.js
diff --git a/chrome/test/data/webui/media_router/media_router_container_filter_tests.js b/chrome/test/data/webui/media_router/media_router_container_filter_tests.js
index df7da7dd18dd2fcad4d8f897bdabeb1de3a9d97c..c85ed5e898862b8ec6ff705832e02cf0d6f71971 100644
--- a/chrome/test/data/webui/media_router/media_router_container_filter_tests.js
+++ b/chrome/test/data/webui/media_router/media_router_container_filter_tests.js
@@ -200,27 +200,29 @@ cr.define('media_router_container_filter', function() {
container.allSinks = fakeSinkList;
MockInteractions.tap(container.$$('#sink-search-icon'));
setTimeout(function() {
- var searchResults = container.$$('#search-results');
+ var searchResultsPaperMenu =
+ container.$$('#search-results-paper-menu');
// Use the Polymer method for selecting (and focusing on) a sink
- searchResults.selectIndex(focusIndex);
+ searchResultsPaperMenu.selectIndex(focusIndex);
var itemInSearch =
- searchResults.querySelectorAll('paper-item')[focusIndex];
+ searchResultsPaperMenu.querySelectorAll('paper-item')[focusIndex];
// TODO(crbug.com/608551): This condition handles flakiness around
// the search item getting focus earlier. If it doesn't get focus,
// the logic that changes focus from a search item to a sink list
// item obviously won't do anything.
if (itemInSearch.focused) {
- var selectedIndexInSearch = searchResults.selected;
+ var selectedIndexInSearch = searchResultsPaperMenu.selected;
pressEscapeOnElement(itemInSearch);
checkCurrentView(media_router.MediaRouterView.SINK_LIST);
chainOnAnimationPromise(function() {
- var sinkList = container.$$('#sink-list');
- var item = sinkList.querySelectorAll('paper-item')[focusIndex];
+ var sinkListPaperMenu = container.$$('#sink-list-paper-menu');
+ var item =
+ sinkListPaperMenu.querySelectorAll('paper-item')[focusIndex];
// Check that the "focused" HTML attribute persists.
assertTrue(item.focused);
// Check that the "selected" Polymer attribute persists.
- assertEquals(sinkList.selected, selectedIndexInSearch);
+ assertEquals(sinkListPaperMenu.selected, selectedIndexInSearch);
done();
});
} else {
« no previous file with comments | « chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698