[MD Bookmarks] Change selection items from Map to Set.

chrome/test/data/webui/md_bookmarks/reducers_test.js

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 suite('selection state', function() {
   var state;
   var action;
 
   function select(items, anchor, add) {
     return {
       name: 'select-items',
       add: add,
       anchor: anchor,
       items: items,
     };
   }
 
   setup(function() {
     state = {
       anchor: null,
       items: {},
     };
   });
 
   test('can select an item', function() {
     action = select(['1'], '1', false);
     state = bookmarks.SelectionState.updateSelection(state, action);
 
-    assertDeepEquals({'1': true}, state.items);
+    assertDeepEquals(['1'], normalizeSet(state.items));
     assertEquals('1', state.anchor);
 
     // Replace current selection.
     action = select(['2'], '2', false);
     state = bookmarks.SelectionState.updateSelection(state, action);
-    assertDeepEquals({'2': true}, state.items);
+    assertDeepEquals(['2'], normalizeSet(state.items));
     assertEquals('2', state.anchor);
 
     // Add to current selection.
     action = select(['3'], '3', true);
     state = bookmarks.SelectionState.updateSelection(state, action);
-    assertDeepEquals({'2': true, '3': true}, state.items);
+    assertDeepEquals(['2', '3'], normalizeSet(state.items));
     assertEquals('3', state.anchor);
   });
 
   test('can select multiple items', function() {
     action = select(['1', '2', '3'], '3', false);
     state = bookmarks.SelectionState.updateSelection(state, action);
-    assertDeepEquals({'1': true, '2': true, '3': true}, state.items);
+    assertDeepEquals(['1', '2', '3'], normalizeSet(state.items));
 
     action = select(['3', '4'], '4', true);
     state = bookmarks.SelectionState.updateSelection(state, action);
-    assertDeepEquals({'1': true, '2': true, '3': true, '4': true}, state.items);
+    assertDeepEquals(['1', '2', '3', '4'], normalizeSet(state.items));
   });
 
   test('is cleared when selected folder changes', function() {
     action = select(['1', '2', '3'], '3', false);
     state = bookmarks.SelectionState.updateSelection(state, action);
 
     action = bookmarks.actions.selectFolder('2');
     state = bookmarks.SelectionState.updateSelection(state, action);
     assertDeepEquals({}, state.items);
   });
@@ skipping 33 matching lines @@
 
   setup(function() {
     nodes = testTree(
         createFolder(
             '1',
             [
               createFolder('2', []),
               createItem('3'),
             ]),
         createFolder('4', []));
-    state = {};
+    state = new Set();
   });
 
   test('toggle folder open state', function() {
     action = bookmarks.actions.changeFolderOpen('2', false);
     state =
         bookmarks.ClosedFolderState.updateClosedFolders(state, action, nodes);
-    assertFalse(!!state['1']);
-    assertTrue(state['2']);
+    assertFalse(state.has('1'));
+    assertTrue(state.has('2'));
   });
 
   test('select folder with closed parent', function() {
     // Close '1'
     action = bookmarks.actions.changeFolderOpen('1', false);
     state =
         bookmarks.ClosedFolderState.updateClosedFolders(state, action, nodes);
-    assertTrue(state['1']);
-    assertFalse(!!state['2']);
+    assertTrue(state.has('1'));
+    assertFalse(state.has('2'));
 
     // Should re-open when '2' is selected.
     action = bookmarks.actions.selectFolder('2');
     state =
         bookmarks.ClosedFolderState.updateClosedFolders(state, action, nodes);
-    assertFalse(!!state['1']);
+    assertFalse(state.has('1'));
   });
 
   test('move nodes in a closed folder', function() {
     // Moving bookmark items should not open folders.
-    state = {'1': true};
+    state = new Set(['1']);
     action = bookmarks.actions.moveBookmark('3', '1', 1, '1', 0);
     state =
         bookmarks.ClosedFolderState.updateClosedFolders(state, action, nodes);
 
-    assertTrue(state['1']);
+    assertTrue(state.has('1'));
 
     // Moving folders should open their parents.
-    state = {'1': true, '2': true};
+    state = new Set(['1', '2']);
     action = bookmarks.actions.moveBookmark('4', '2', 0, '0', 1);
     state =
         bookmarks.ClosedFolderState.updateClosedFolders(state, action, nodes);
-    assertFalse(!!state['1']);
-    assertFalse(!!state['2']);
+    assertFalse(state.has('1'));
+    assertFalse(state.has('2'));
   });
 });
 
 suite('selected folder', function() {
   var nodes;
   var state;
   var action;
 
   setup(function() {
     nodes = testTree(createFolder('1', [
@@ skipping 150 matching lines @@
     // Case 2: Clear search by selecting a new folder.
     action = bookmarks.actions.selectFolder('2');
     var selectedState = bookmarks.reduceAction(searchedState, action);
 
     assertEquals('2', selectedState.selectedFolder);
     assertDeepEquals(['3'], bookmarks.util.getDisplayedList(selectedState));
     assertEquals('', selectedState.search.term);
     assertDeepEquals([], selectedState.search.results);
   });
 });