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

Side by Side Diff: chrome/browser/resources/bookmark_manager/js/main.js

Issue 835973004: Fix for 'New Folder' is not getting added to chrome://bookmarks/ through 'Organize' menu after addi… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 (function() { 5 (function() {
6 /** @const */ var BookmarkList = bmm.BookmarkList; 6 /** @const */ var BookmarkList = bmm.BookmarkList;
7 /** @const */ var BookmarkTree = bmm.BookmarkTree; 7 /** @const */ var BookmarkTree = bmm.BookmarkTree;
8 /** @const */ var Command = cr.ui.Command; 8 /** @const */ var Command = cr.ui.Command;
9 /** @const */ var LinkKind = cr.LinkKind; 9 /** @const */ var LinkKind = cr.LinkKind;
10 /** @const */ var ListItem = cr.ui.ListItem; 10 /** @const */ var ListItem = cr.ui.ListItem;
(...skipping 1016 matching lines...) Expand 10 before | Expand all | Expand 10 after
1027 */ 1027 */
1028 function newFolder(opt_target) { 1028 function newFolder(opt_target) {
1029 performGlobalUndo = null; // This can't be undone, so disable global undo. 1029 performGlobalUndo = null; // This can't be undone, so disable global undo.
1030 1030
1031 var parentId = computeParentFolderForNewItem(); 1031 var parentId = computeParentFolderForNewItem();
1032 var selectedItem = bmm.list.selectedItem; 1032 var selectedItem = bmm.list.selectedItem;
1033 var newIndex; 1033 var newIndex;
1034 // Callback is called after tree and list data model updated. 1034 // Callback is called after tree and list data model updated.
1035 function createFolder(callback) { 1035 function createFolder(callback) {
1036 if (selectedItem && document.activeElement != bmm.tree && 1036 if (selectedItem && document.activeElement != bmm.tree &&
1037 !bmm.isFolder(selectedItem)) { 1037 !bmm.isFolder(selectedItem) && selectedItem.id != 'new') {
1038 newIndex = bmm.list.dataModel.indexOf(selectedItem) + 1; 1038 newIndex = bmm.list.dataModel.indexOf(selectedItem) + 1;
1039 } 1039 }
1040 chrome.bookmarks.create({ 1040 chrome.bookmarks.create({
1041 title: loadTimeData.getString('new_folder_name'), 1041 title: loadTimeData.getString('new_folder_name'),
1042 parentId: parentId, 1042 parentId: parentId,
1043 index: newIndex 1043 index: newIndex
1044 }, callback); 1044 }, callback);
1045 } 1045 }
1046 1046
1047 if ((opt_target || document.activeElement) == bmm.tree) { 1047 if ((opt_target || document.activeElement) == bmm.tree) {
(...skipping 458 matching lines...) Expand 10 before | Expand all | Expand 10 after
1506 1506
1507 cr.ui.FocusOutlineManager.forDocument(document); 1507 cr.ui.FocusOutlineManager.forDocument(document);
1508 initializeSplitter(); 1508 initializeSplitter();
1509 bmm.addBookmarkModelListeners(); 1509 bmm.addBookmarkModelListeners();
1510 dnd.init(selectItemsAfterUserAction); 1510 dnd.init(selectItemsAfterUserAction);
1511 bmm.tree.reload(); 1511 bmm.tree.reload();
1512 } 1512 }
1513 1513
1514 initializeBookmarkManager(); 1514 initializeBookmarkManager();
1515 })(); 1515 })();
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698