Closed Bug 1678624 Opened 1 year ago Closed 2 months ago

Convert onItemChanged("uri") to a bookmark-url-changed PlacesEvent

Categories

(Toolkit :: Places, task, P3)

task
Points:
3

Tracking

()

RESOLVED FIXED
95 Branch
Iteration:
87.2 - Feb 8 - Feb 21
Tracking Status
firefox95 --- fixed

People

(Reporter: mak, Assigned: daisuke)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

onItemChanged() for "uri" can be converted into its own event, derived from PlacesBookmark.

Assignee: nobody → daisuke
Status: NEW → ASSIGNED
Iteration: --- → 87.2 - Feb 8 - Feb 21
Points: --- → 3

Depends on D103883

Same question as for bug 1678623, are these just wip patches? I'm not trying to speed it up, just ensuring not setting a reviewer was not a mistake.

Flags: needinfo?(daisuke)

Yep, thank you very much.
I have requested review because it seems that rebasing was successful and worked well.
https://treeherder.mozilla.org/jobs?repo=try&revision=931e66e4fe2b9ff6fe8ae3d24fdad59769c3a52d

Flags: needinfo?(daisuke)
Blocks: 1734048
Pushed by dakatsuka.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/78dd2fb234ee
Implement a mechanism to fire bookmark-url-changed event. r=mak
https://hg.mozilla.org/integration/autoland/rev/cb7432a73c16
Apply bookmark-url-changed event. r=mak
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.