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

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

Issue 2319123002: Profile Avatar Selector: Allow arrow keys to be used for moving between avatars (2nd edition) (Closed)
Patch Set: 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
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..190d4edee9ef002967e4d674ae8f8c4738e7c82f 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-list'].items.length);
});
test('Can update avatars', function() {
avatarSelector.pop('avatars');
Polymer.dom.flush();
- assertEquals(2, avatarSelector.$.selector.items.length);
+ assertEquals(2, avatarSelector.$['avatar-list'].items.length);
});
test('No avatar is initially selected', function() {
- var selector = avatarSelector.$.selector;
+ var selector = avatarSelector.$['avatar-list'];
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-list'];
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-list'];
// 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-list'].items[1]);
});
});
}

Powered by Google App Engine
This is Rietveld 408576698