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

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

Issue 2617943002: [MD Bookmarks] Edit Bookmarks. (Closed)
Patch Set: style fix Created 3 years, 11 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/md_bookmarks/store_test.js
diff --git a/chrome/test/data/webui/md_bookmarks/store_test.js b/chrome/test/data/webui/md_bookmarks/store_test.js
index 8b9099f329e4430e50e9f279220833e61c9004d6..ae0772a262dec3d057bddd1040e46b84198d2eb6 100644
--- a/chrome/test/data/webui/md_bookmarks/store_test.js
+++ b/chrome/test/data/webui/md_bookmarks/store_test.js
@@ -171,4 +171,24 @@ suite('<bookmarks-store>', function() {
assertTrue(store.idToNodeMap_['0'].isSelected);
assertEquals('0', store.selectedId);
});
+
+ test('bookmark gets updated after editing', function() {
+ // Edit title updates idToNodeMap_ properly.
+ store.onBookmarkChanged_('4', {'title': 'test'});
+ assertEquals('test', store.idToNodeMap_['4'].title);
+ assertEquals('link4', store.idToNodeMap_['4'].url);
+
+ // Edit url updates idToNodeMap_ properly.
+ store.onBookmarkChanged_('5', {'url': 'http://www.google.com'});
+ assertEquals(undefined, store.idToNodeMap_['5'].title);
+ assertEquals('http://www.google.com', store.idToNodeMap_['5'].url);
+
+ // Edit url and title updates idToNodeMap_ properly.
+ store.onBookmarkChanged_('2', {
+ 'title': 'test',
+ 'url': 'http://www.google.com',
+ });
+ assertEquals('test', store.idToNodeMap_['2'].title);
+ assertEquals('http://www.google.com', store.idToNodeMap_['2'].url);
+ });
});

Powered by Google App Engine
This is Rietveld 408576698