Remove unused code from #11040

Auditors: @NejcZdovc
brave · Oct 3, 2017 · 9cfeae0 · 9cfeae0 · NejcZdovc · Oct 3, 2017
1 parent e357db4
commit 9cfeae0
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 12 deletions.
diff --git a/app/common/state/bookmarksState.js b/app/common/state/bookmarksState.js
@@ -41,10 +41,6 @@ const bookmarksState = {
     return state.getIn([STATE_SITES.BOOKMARKS, key], Immutable.Map())
   },
 
-  getBookmarkOrder: (state) => {
-    return state.getIn(STATE_SITES.BOOKMARK_ORDER_PATH)
-  },
-
   /**
    * Use this function if you only have a key and don't know if key is for folder or regular bookmark
    * @param state

diff --git a/js/constants/stateConstants.js b/js/constants/stateConstants.js
@@ -6,8 +6,7 @@ const STATE_SITES = {
   BOOKMARKS: 'bookmarks',
   BOOKMARK_FOLDERS: 'bookmarkFolders',
   HISTORY_SITES: 'historySites',
-  PINNED_SITES: 'pinnedSites',
-  BOOKMARK_ORDER_PATH: ['cache', 'bookmarkOrder']
+  PINNED_SITES: 'pinnedSites'
 }
 
 module.exports = {

diff --git a/test/unit/app/common/state/bookmarksStateTest.js b/test/unit/app/common/state/bookmarksStateTest.js
@@ -66,12 +66,6 @@ const stateWithData = Immutable.fromJS({
 })
 
 describe('bookmarkState unit test', function () {
-  describe('getBookmarkOrder', function () {
-    it('resturns order state', function () {
-      assert.deepEqual(bookmarksState.getBookmarkOrder(stateWithData), stateWithData.getIn(['cache', 'bookmarkOrder']))
-    })
-  })
-
   describe('updateFavicon', function () {
     it('updates the favicon for all matching entries', function () {
       const processedState = bookmarksState.updateFavicon(stateWithData, 'https://brave.com/', 'https://brave.com/favicon.ico')