[Bug 27874] TpBaseClient Approver support

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=27874

--- Comment #10 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-05-07 03:57:00 PDT ---
(In reply to comment #9)
> Why not just do the same trick in the CDO using
> _tp_dbus_daemon_is_the_shared_one and create the AM accordingly? That way we
> wouldn't have to add new API.

You unavoidably end up with a different TpAccount and TpConnection in all
non-shared-bus cases, which seems less than ideal.

> Note that this doesn't solve the "issue" of having different TpChannel objects
> in the context and in CDO.

Yes, that's a fair point; perhaps the approver context should be able to stuff
TpChannel objects into the CDO, too (maybe with library-internal API).

For Observers that are acting as non-interactive approvers, it would also be
useful for the observer context to be able to pre-populate the CDO with the
right TpAccount, TpConnection and initial TpChannel objects; the CDO still
wouldn't be pre-prepared, but when asked to prepare, it would use its existing
objects.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.