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

Status: RESOLVED FIXED

(Toolkit :: Places, task, P3)

Description

[Marco Bonardo [:mak]]

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

Comment 1

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1678624: Implement a mechanism to fire bookmark-url-changed event.

Comment 2

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1678624: Apply bookmark-url-changed event.

Depends on D103883

Comment 3

[Marco Bonardo [:mak]]

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.

Comment 4

[Daisuke Akatsuka (: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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
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

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/78dd2fb234ee
https://hg.mozilla.org/mozilla-central/rev/cb7432a73c16