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

Unified Diff: chrome/test/data/webui/md_bookmarks/sidebar_test.js

Issue 2812493002: MD Bookmarks: Restore any previously selected folder when clearing search (Closed)
Patch Set: Fix comment Created 3 years, 8 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/test/data/webui/md_bookmarks/router_test.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/md_bookmarks/sidebar_test.js
diff --git a/chrome/test/data/webui/md_bookmarks/sidebar_test.js b/chrome/test/data/webui/md_bookmarks/sidebar_test.js
index 79066696e8f09954fc5c46a11a1aa00fd3bbe3e9..097ba79e27d48bbffc1bb574c509606f58de95be 100644
--- a/chrome/test/data/webui/md_bookmarks/sidebar_test.js
+++ b/chrome/test/data/webui/md_bookmarks/sidebar_test.js
@@ -70,5 +70,25 @@ suite('<bookmarks-sidebar>', function() {
assertEquals(2, f.depth);
assertEquals('2', f.style.getPropertyValue('--node-depth'));
});
- })
+ });
+
+ test('doesn\'t highlight selected folder while searching', function() {
+ var rootFolders =
+ sidebar.$['folder-tree'].querySelectorAll('bookmarks-folder-node');
+
+ store.data.selectedFolder = '1';
+ store.notifyObservers();
+
+ assertEquals('1', rootFolders['0'].itemId);
+ assertTrue(rootFolders['0'].isSelectedFolder_);
+
+ store.data.search = {
+ term: 'test',
+ inProgress: false,
+ results: ['3'],
+ };
+ store.notifyObservers();
+
+ assertFalse(rootFolders['0'].isSelectedFolder_);
+ });
});
« no previous file with comments | « chrome/test/data/webui/md_bookmarks/router_test.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698