Merge pull request #9432 from brave/fix/over-sorting-sites

Don't sort sites when adding or removing them
brave · Jun 14, 2017 · 89058cf · 89058cf
2 parents c5718d0 + 04f1313
commit 89058cf
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/app/browser/reducers/sitesReducer.js b/app/browser/reducers/sitesReducer.js
@@ -75,12 +75,10 @@ const sitesReducer = (state, action, immutableAction) => {
           state = syncUtil.updateSiteCache(state, action.destinationDetail || action.siteDetail)
         }
       }
-      state = state.set('sites', state.get('sites').sort(siteUtil.siteSort))
       state = updateActiveTabBookmarked(state)
       break
     case appConstants.APP_REMOVE_SITE:
       state = siteUtil.removeSite(state, action.siteDetail, action.tag, true)
-      state = state.set('sites', state.get('sites').sort(siteUtil.siteSort))
       if (syncEnabled()) {
         state = syncUtil.updateSiteCache(state, action.siteDetail)
       }

diff --git a/app/sync.js b/app/sync.js
@@ -266,7 +266,6 @@ module.exports.onSyncReady = (isFirstRun, e) => {
 
   // Sync bookmarks that have not been synced yet.
   siteUtil.getBookmarks(sites).filter(site => shouldSyncBookmark(site))
-    .sortBy(site => site.get('order'))
     .forEach(syncBookmark)
 
   // Sync site settings that have not been synced yet