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

Unified Diff: chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js

Issue 2269743002: Profile Avatar Selector: Allow arrow keys to be used for moving between avatars. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments 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
« no previous file with comments | « no previous file | chrome/test/data/webui/settings/people_page_manage_profile_test.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js
diff --git a/chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js b/chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js
index 61ae1637cf06f7b899301b7a6b24a7876d97e08e..d42299705cad1a354ede205ab1e852887230ef85 100644
--- a/chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js
+++ b/chrome/test/data/webui/cr_elements/cr_profile_avatar_selector_tests.js
@@ -31,17 +31,17 @@ cr.define('cr_profile_avatar_selector', function() {
});
test('Displays avatars', function() {
- assertEquals(3, avatarSelector.$.selector.items.length);
+ assertEquals(3, avatarSelector.$['avatar-grid'].items.length);
});
test('Can update avatars', function() {
avatarSelector.pop('avatars');
Polymer.dom.flush();
- assertEquals(2, avatarSelector.$.selector.items.length);
+ assertEquals(2, avatarSelector.$['avatar-grid'].items.length);
});
test('No avatar is initially selected', function() {
- var selector = avatarSelector.$.selector;
+ var selector = avatarSelector.$['avatar-grid'];
assertFalse(!!avatarSelector.selectedAvatarUrl);
assertFalse(selector.items[0].classList.contains('iron-selected'));
@@ -57,7 +57,7 @@ cr.define('cr_profile_avatar_selector', function() {
document.body.appendChild(anotherAvatarSelector);
Polymer.dom.flush();
- var selector = anotherAvatarSelector.$.selector;
+ var selector = anotherAvatarSelector.$['avatar-grid'];
assertEquals('chrome://avatar1.png',
anotherAvatarSelector.selectedAvatarUrl);
@@ -67,7 +67,7 @@ cr.define('cr_profile_avatar_selector', function() {
});
test('Can select avatar', function() {
- var selector = avatarSelector.$.selector;
+ var selector = avatarSelector.$['avatar-grid'];
// Simulate tapping the third avatar.
MockInteractions.tap(selector.items[2]);
@@ -86,7 +86,7 @@ cr.define('cr_profile_avatar_selector', function() {
});
// Simulate tapping the second avatar.
- MockInteractions.tap(avatarSelector.$.selector.items[1]);
+ MockInteractions.tap(avatarSelector.$['avatar-grid'].items[1]);
});
});
}
« no previous file with comments | « no previous file | chrome/test/data/webui/settings/people_page_manage_profile_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698