Convert consumers of removePagesByTimeframe and removePagesFromHost to PlacesUtils.history.removeByFilter

NEW
Unassigned

Status

()

Toolkit
Places
P2
normal
3 years ago
6 months ago

People

(Reporter: Yoric, Unassigned, Mentored)

Tracking

(Blocks: 1 bug, {good-first-bug, perf})

unspecified
good-first-bug, perf
Points:
5
Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][lang=js])

Comment hidden (empty)
Points: --- → 5
Flags: qe-verify-
Flags: firefox-backlog+
Blocks: 739219
Priority: -- → P2
Summary: Convert tests using removeVisitsByTimeframe, removePagesByHost to removeByFilter → Convert tests using removePagesByTimeframe, removePagesByHost to remove
Summary: Convert tests using removePagesByTimeframe, removePagesByHost to remove → Convert tests using removePagesByTimeframe, removePagesFromHost to remove
Depends on: 1249263
this needs bug 1249263 first.
Summary: Convert tests using removePagesByTimeframe, removePagesFromHost to remove → Convert tests using removePagesByTimeframe, removePagesFromHost to removeByFilter
Summary: Convert tests using removePagesByTimeframe, removePagesFromHost to removeByFilter → Convert consumers of removePagesByTimeframe and removePagesFromHost to PlacesUtils.history.removeByFilter
And this is now actionable!
The only thing to take into account is that the new API is asynchronous, so the call points must be adapted to use promises
Mentor: mak77@bonardo.net
Keywords: good-first-bug
Whiteboard: [good first bug][lang=js]
http://searchfox.org/mozilla-central/search?q=%5C.removePages%28By%7CFromH%29&regexp=true&path=
can i work on this?
Sure, do you have a working build already? An artifact build should be enough.
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction
Keywords: perf
You need to log in before you can comment on or make changes to this bug.