885251 - Don't use getPageTitle in PlacesTagURITransaction

Status: RESOLVED WONTFIX

(Toolkit :: Places, defect)

Description

[Asaf Romano (gone)]

______________
So... what's the story for async transaction managers? I'm pretty sure nsITransactionManager assumes synchronous implementations in many ways. Unfortunately this issue applies to all places transactions. This bug won't cover that.
______________

PlacesTagURITransaction is using getPageTitle. It shouldn't. If the issue mentioned above isn't major, it could be swiitched to getPlacesInfo right away.  If it is, then its caller would have to provide it with the title. Alternatively, we could avoid storing titles for these hacky-bookmarks and always show the title from history.

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

I'm not sure how we will handle the transactions, but likely we won't be able to just use nsITransactionManager directly as we do today, but we'll have to wrap everything into a new transactions module with its own serialized API.

Comment 2

[Marco Mucci [:MarcoM]]

Team determined that this bug will be 'Wonfix' at the point estimation meeting.

Comment 3

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

the reason is that with async transactions we will not use this api, so there's no reason to update the old transactions.