Closed Bug 1473530 Opened 6 years ago Closed 2 years ago

[meta] Convert bookmark notifications to the new system

Categories

(Toolkit :: Places, enhancement, P3)

Product:
Toolkit
Component:
Places
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
112 Branch

People

(Reporter: mak, Unassigned)

References

(Blocks 7 open bugs)

Details

(Keywords: meta, perf, Whiteboard: [fxsearch])

+++ This bug was initially created as a clone of Bug #1473529 +++ To take advantage of the new notifications, we must convert the existing ones to the new system and fix consumers to properly handle batches. We should replace all the notifications in nsINavBookmarkObserver, we can do that in chunks in dependent bugs.
No longer blocks: 1426245
Depends on: 1426245
Blocks: 1405242
Summary: Convert bookmark notifications to the new system → [meta] Convert bookmark notifications to the new system
Depends on: 1511346
Depends on: 1511062
Depends on: 1511700
Blocks: 1533061
Blocks: 1562490
Depends on: 1607245
Blocks: 1678605
Depends on: 1678607
No longer depends on: 424160
Depends on: 1677408
Depends on: 1678623
Depends on: 1678624
Depends on: 1678626
Depends on: 1678627
Depends on: 1678628
Depends on: 1678629
No longer blocks: 1733045
Depends on: 1733045
Depends on: 1733854
Depends on: 1749900
Depends on: 1764776
Severity: normal → S3
Depends on: 1744866
Duplicate of this bug: 1678605

Theres'a few leftovers, and then we can resolve this:
https://searchfox.org/mozilla-central/search?q=nsiNav%28history%7Cbookmark%29Observer&path=&case=false&regexp=true

Flags: needinfo?(daisuke)
Depends on: 1818119

Thank you, Marco. I will do it in bug 1818119.

Flags: needinfo?(daisuke)

This was resolved by dependencies, thanks to everyone involved.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Blocks: 1821664
You need to log in before you can comment on or make changes to this bug.