Closed
Bug 1382991
Opened 7 years ago
Closed 7 years ago
Cleanup InsertionPoint once we reach 57
Categories
(Firefox :: Bookmarks & History, enhancement, P2)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
Firefox 57
| Tracking | Status | |
|---|---|---|
| firefox57 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
InsertionPoint has a lot of compatibility shims and horrible API, we should clean it up.
|
Assignee | |
Updated•7 years ago
|
Blocks: placesAsyncBookmarks
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → mak77
Status: NEW → ASSIGNED
| Comment hidden (mozreview-request) |
|
||
Comment 2•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8893352 [details]
Bug 1382991 - Cleanup Places controller InsertionPoint.
https://reviewboard.mozilla.org/r/164442/#review169746
Looks good, much nicer!
Attachment #8893352 -
Flags: review?(standard8) → review+
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/105df91f59e5
Cleanup Places controller InsertionPoint. r=standard8
|
||
Comment 4•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in
before you can comment on or make changes to this bug.
Description
•