Closed
Bug 1475276
Opened 6 years ago
Closed 6 years ago
Remove some no more necessary usage of promiseItemGuid
Categories
(Toolkit :: Places, enhancement, P2)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
Long term goal is to completely remove promiseItemGuid from production code, and move it to PlacesTestUtils.
For now we can't do that because of livemarks, but this is a first step towards that.
|
Assignee | |
Comment 1•6 years ago
|
||
Some code is unnecessarily using promiseItemGuid
|
|
Assignee | |
Comment 2•6 years ago
|
||
This includes a small change to NewTabUtils that will have to be ported to github, cc-ing ursula.
|
||
Comment 3•6 years ago
|
||
Comment on attachment 8991659 [details]
Bug 1475276 - Remove some no more necessary usage of promiseItemGuid. r=lina
Lina Cambridge (she/her) [:lina, :kitcambridge] has approved the revision.
https://phabricator.services.mozilla.com/D2100
Attachment #8991659 -
Flags: review+
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/731d1bf09a74
Remove some no more necessary usage of promiseItemGuid. r=lina
|
||
Comment 5•6 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•