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

Unified Diff: chrome/browser/resources/md_bookmarks/dnd_manager.js

Issue 2776993002: [MD Bookmarks] Persist collapsed folders between page loads. (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/md_bookmarks/dnd_manager.js
diff --git a/chrome/browser/resources/md_bookmarks/dnd_manager.js b/chrome/browser/resources/md_bookmarks/dnd_manager.js
index 19a6f889a5eed7a467dc13de317f6b18607e5554..fb2a86d9149e9246ee0efd3cf8248854b3c0111e 100644
--- a/chrome/browser/resources/md_bookmarks/dnd_manager.js
+++ b/chrome/browser/resources/md_bookmarks/dnd_manager.js
@@ -2,22 +2,6 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-/**
- * Enumeration of valid drop locations relative to an element. These are
- * bit masks to allow combining multiple locations in a single value.
- * @enum {number}
- * @const
- */
-var DropPosition = {
- NONE: 0,
- ABOVE: 1,
- ON: 2,
- BELOW: 4,
-};
-
-/** @typedef {{element: BookmarkElement, position: DropPosition}} */
-var DropDestination;
-
cr.define('bookmarks', function() {
/**
* @param {BookmarkElement} element
@@ -497,7 +481,7 @@ cr.define('bookmarks', function() {
var state = bookmarks.Store.getInstance().data;
// We cannot drop between Bookmarks bar and Other bookmarks.
- if (getBookmarkNode(overElement).parentId == bookmarks.util.ROOT_NODE_ID)
+ if (getBookmarkNode(overElement).parentId == ROOT_NODE_ID)
return DropPosition.NONE;
var isOverFolderNode = isBookmarkFolderNode(overElement);

Powered by Google App Engine
This is Rietveld 408576698