Closed Bug 1678606 Opened 4 years ago Closed 4 years ago

Convert onClearHistory to a new history-cleared PlacesEvent

Categories

(Toolkit :: Places, task, P3)

task
Points:
3

Tracking

()

RESOLVED FIXED
86 Branch
Iteration:
85.3 - Dec 14 - Dec 27
Tracking Status
firefox86 --- fixed

People

(Reporter: mak, Assigned: daisuke)

References

Details

(Keywords: perf)

Attachments

(3 files)

A few consumers are listening to history changes just for onClearHistory, and that's a waste of resources, we should provide a direct history-cleared notification instead.

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Iteration: --- → 85.3 - Dec 14 - Dec 27
Points: --- → 3

Depends on D99424

Pushed by dakatsuka.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1cf7a9b74ce9 Imlement a mechanism to fire history-cleared event. r=mak https://hg.mozilla.org/integration/autoland/rev/dd486bf6bbb8 Apply history-cleared event instead of onClearHistory. r=mixedpuppy,mak https://hg.mozilla.org/integration/autoland/rev/87e34c325a1d Remove onClearHistory interface from nsINavHistoryService. r=mixedpuppy,mak
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: