Remove the ability to sort results by annotation

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P1
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: mak, Assigned: standard8)

Tracking

(Blocks 1 bug)

unspecified
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

(Whiteboard: [fxsearch])

Attachments

(1 attachment)

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
Blocks: 1460577
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 mbanner@mozilla.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
https://hg.mozilla.org/mozilla-central/rev/30a61cd82d54
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.