Remove PlacesUtils.asyncHistory

RESOLVED FIXED in Firefox 61

Status

()

P3
normal
RESOLVED FIXED
2 years ago
10 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)

(Assignee)

Description

2 years ago
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.
(Assignee)

Updated

2 years ago
Depends on: 1350377
(Assignee)

Updated

2 years ago
Depends on: 1366231
(Assignee)

Updated

2 years ago
Depends on: 1370881
Depends on: 1448041
(Assignee)

Updated

11 months ago
Blocks: 1455079
(Assignee)

Comment 1

11 months ago
we can likely complete this now.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
(Assignee)

Updated

11 months ago
Depends on: 1458865
(Assignee)

Updated

11 months ago
Blocks: 1458865
No longer depends on: 1458865
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 5

11 months 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

11 months 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

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/718d6eca69f2
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
(Assignee)

Updated

11 months ago
Blocks: 1459855
status-firefox55: affected → ---
You need to log in before you can comment on or make changes to this bug.