[Telepathy] [Bug 12465] Current avatars API is likely to generate too much DBus traffic

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=12465





------- Comment #7 from xclaesse at gmail.com  2007-11-26 02:04 PST -------
> And it wouldn't be too bad to add the avatar data in the Avatar(s)Updated
> signal itself, since most listeners of this signal would also want to retrieve
> the avatar, once they see that the token has changed. In this way the
> RequestAvatars method calls would probably be more rare.

(Here I say "Empathy" but it can be any tp client)

RequestAvatars is already extremely rare, Empathy cache avatars on file, once
Empathy gets the token it gets the avatar from the cache most time and don't
call RequestAvatars. If empathy don't have the avatar in cache it's most likely
that all processes listening to AvatarRetrieved would need the avatar data
anyway.

MC is a bit special here since it only needs self avatar so if it listen to
AvatarRetrieved it will get all avatars. It can be optimised by connecting this
signal only when MC receives AvatarUpdated(self handle) with a different token
than the one currently stored, then call RequestAvatars(self handle) and
disconnect the AvatarRetrieved once MC gets new self avatar data.

And CM should wait a bit when receiving RequestAvatars() because it's likely to
get one request from many clients at once, so it could emit AvatarRetrieved
only once with all avatars requested.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.