Open Bug 1408364 Opened 7 years ago Updated 2 years ago

Consider using PlacesUtils.history.fetch to implement history.getVisits

Categories

(WebExtensions :: General, enhancement, P3)

enhancement

Tracking

(firefox57 wontfix)

Tracking Status
firefox57 --- wontfix

People

(Reporter: emk, Unassigned)

References

Details

Quoting from bug 1406594 comment #11:
> A separate question though, could getVisits use PlacesUtils.history.fetch
> instead, with the includeVisits option? That API is async and thus better,
> though it may be missing some of the properties you are looking for. In
> particular visitId and referringVisitId. We could expose that info, even if
> I was honestly trying to stop exposing visit ids. An API should never expose
> raw database details. And I don't see where these ids are used by the
> chrome.history API, I cannot find any method that takes a VisitItem or a
> visit id. A visit can be completely identified by url, transition and time.
> Btw, it's a possibility we can evaluate.
Severity: normal → enhancement
Priority: -- → P3
Product: Toolkit → WebExtensions
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.