Closed
Bug 1354531
Opened 8 years ago
Closed 7 years ago
Remove PlacesUtils.asyncHistory
Categories
(Toolkit :: Places, enhancement, P3)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla61
| Tracking | Status | |
|---|---|---|
| firefox61 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
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 | |
Comment 1•7 years ago
|
||
we can likely complete this now.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Updated•7 years ago
|
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 5•7 years 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+
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•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
|
||
Updated•6 years ago
|
status-firefox55:
affected → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•