Closed Bug 1465347 Opened 2 years ago Closed 2 years ago
Remove the ability to sort results by annotation
46 bytes, text/x-phabricator-request
|Details | Review|
nsINavHistoryResult::sortingAnnotation and SORT_BY_ANNOTATION_* can go away along with all the code supporting them. This is a nice simplification since it may allow us to move all the sorting to SQL in the future. PS: there's a "newSortingAnnotation" var leftover in treeView.js
I was going to do this as part of bug 1402890, however, having a separate reminder won't hurt.
we can do this sooner than bug 1402890.
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Priority: P2 → P1
Try push is pending tree re-opening.
Comment on attachment 8985439 [details] Bug 1465347 - Remove the ability to sort results by annotation as it is no longer used. Marco Bonardo [::mak] has approved the revision. https://phabricator.services.mozilla.com/D1662
Attachment #8985439 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/30a61cd82d54 Remove the ability to sort results by annotation as it is no longer used. r=mak
You need to log in before you can comment on or make changes to this bug.