Remove nsINavBookmarkObserver
Categories
(Toolkit :: Places, task, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox112 | --- | fixed |
People
(Reporter: daisuke, Assigned: daisuke)
References
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
text/x-phabricator-request
|
Details |
This is a follow-up bug for bug 1678624 especially Marco's comment in https://phabricator.services.mozilla.com/D103148#4110377.
After we replace all event system with new, we should remove nsINavBookmarkObserver.
https://searchfox.org/mozilla-central/rev/79f93e7a8b9aa1903f1349f2dd46fb71596f2ae9/toolkit/components/places/nsINavBookmarksService.idl#17
|
||
Comment 1•3 years ago
|
||
I think this is part of the changes to replace the old system, thus inverting depends/blocks.
|
|
||
Comment 2•3 years ago
|
||
Also rename test_nsINavBookmarkObserver.js
|
|
||
Updated•3 years ago
|
|
Assignee | |
Updated•3 years ago
|
|
|
Assignee | |
Comment 3•3 years ago
|
||
|
|
||
Comment 4•3 years ago
|
||
Daisuke, could you please mark as dependencies all the bugs that should land before this one can?
|
|
Assignee | |
Comment 5•3 years ago
|
||
I'm sorry for my delay. I did it.
|
||
Comment 6•2 years ago
|
||
The following patch is waiting for review from an inactive reviewer:
| ID | Title | Author | Reviewer Status |
|---|---|---|---|
| D131754 | Bug 1733045: Remove nsINavBookmarkObserver | daisuke | mak: Back Aug 21, 2022 |
:daisuke, could you please find another reviewer or abandon the patch if it is no longer relevant?
For more information, please visit auto_nag documentation.
|
|
Assignee | |
Updated•2 years ago
|
|
|
Assignee | |
Comment 7•2 years ago
|
||
|
||
Updated•2 years ago
|
|
|
||
Updated•2 years ago
|
|
||
Comment 9•2 years ago
|
||
| bugherder | ||
Description
•