1473533 - Change the nsINavHistoryObserver::OnDeleteVisits visit time parameter to be a boolean for partial removal

Status: RESOLVED FIXED

(Toolkit :: Places, enhancement, P1)

Description

[Mark Banner (:standard8)]

The current `aVisitTime` parameter to nsINavHistoryObserver::OnDeleteVisits is really only used as an indiciator for if visits have been partially or fully removed.

We should therefore update the API to make that clarification.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1473533 - Change the nsINavHistoryObserver::OnDeleteVisits visit time parameter to be a boolean for partial removal.

MozReview-Commit-ID: YbkHjOv7FH

Comment 2

[Mark Banner (:standard8)]

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2016f6a905eff66ab954b1506608385003ff608a

Comment 3

[Phabricator Automation]

Comment on attachment 8989959 [details]
Bug 1473533 - Change the nsINavHistoryObserver::OnDeleteVisits visit time parameter to be a boolean for partial removal.

Marco Bonardo [::mak] has approved the revision.

https://phabricator.services.mozilla.com/D1970

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9cd9ca34cf92
Change the nsINavHistoryObserver::OnDeleteVisits visit time parameter to be a boolean for partial removal. r=mak

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/9cd9ca34cf92