Remove PlacesUtils.asyncHistory

RESOLVED FIXED in Firefox 61

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
11 months ago

People

(Reporter: mak, Assigned: mak)

Tracking

(Blocks 2 bugs)

Trunk
mozilla61
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment)

This service should only be used internally by Places, so it's worth removing it from PlacesUtils and manually instantiating the service only where needed to reduce confusion.

Consumers should instead keep using PlacesUtils.history.
Depends on: 1350377
Depends on: 1366231
Depends on: 1370881
Depends on: 1448041
Blocks: 1455079
we can likely complete this now.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Depends on: 1458865
Blocks: 1458865
No longer depends on: 1458865
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 5

a year ago
mozreview-review
Comment on attachment 8972863 [details]
Bug 1354531 - Remove PlacesUtils.asyncHistory to make History internals access less convenient.

https://reviewboard.mozilla.org/r/241420/#review247300
Attachment #8972863 - Flags: review?(standard8) → review+

Comment 6

a year ago
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/718d6eca69f2
Remove PlacesUtils.asyncHistory to make History internals access less convenient. r=standard8

Comment 7

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/718d6eca69f2
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Blocks: 1459855
You need to log in before you can comment on or make changes to this bug.