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

Side by Side Diff: chrome/browser/resources/md_bookmarks/util.js

Issue 2746363013: [MD Bookmarks] Add a drag and drop indicator to bookmarks. (Closed)
Patch Set: Created 3 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview Utility functions for the Bookmarks page. 6 * @fileoverview Utility functions for the Bookmarks page.
7 */ 7 */
8 8
9 cr.define('bookmarks.util', function() { 9 cr.define('bookmarks.util', function() {
10 /** 10 /**
11 * @param {!BookmarksPageState} state 11 * @param {!BookmarksPageState} state
12 * @return {!Array<string>} 12 * @return {!Array<string>}
13 */ 13 */
14 function getDisplayedList(state) { 14 function getDisplayedList(state) {
15 if (state.selectedFolder) 15 if (state.selectedFolder)
tsergeant 2017/03/20 03:01:50 Call isShowingSearch from here, so we don't duplic
calamity 2017/03/22 05:29:52 Done.
16 return assert(state.nodes[state.selectedFolder].children); 16 return assert(state.nodes[state.selectedFolder].children);
17 17
18 return state.search.results; 18 return state.search.results;
19 } 19 }
20 20
21 /** 21 /**
22 * @param {BookmarkTreeNode} rootNode 22 * @param {BookmarkTreeNode} rootNode
23 * @return {NodeList} 23 * @return {NodeList}
24 */ 24 */
25 function normalizeNodes(rootNode) { 25 function normalizeNodes(rootNode) {
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 inProgress: false, 60 inProgress: false,
61 results: [], 61 results: [],
62 }, 62 },
63 selection: { 63 selection: {
64 items: {}, 64 items: {},
65 anchor: null, 65 anchor: null,
66 }, 66 },
67 }; 67 };
68 } 68 }
69 69
70 function isShowingSearch(state) {
tsergeant 2017/03/20 03:01:50 Annotate this function
calamity 2017/03/22 05:29:52 Done.
71 return !state.selectedFolder;
72 }
73
70 return { 74 return {
71 createEmptyState: createEmptyState, 75 createEmptyState: createEmptyState,
72 getDisplayedList: getDisplayedList, 76 getDisplayedList: getDisplayedList,
77 isShowingSearch: isShowingSearch,
73 normalizeNodes: normalizeNodes, 78 normalizeNodes: normalizeNodes,
74 }; 79 };
75 }); 80 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698