Bug 34025

Summary: Don't call ObserverChannels with unapproved channels on recovering Observer
Product: Telepathy Reporter: Guillaume Desmottes <guillaume.desmottes>
Component: mission-controlAssignee: Simon McVittie <smcv>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium CC: vivek
Version: unspecified   
Hardware: Other   
OS: All   
URL: http://cgit.collabora.com/git/user/smcv/telepathy-mission-control-smcv.git/log/?h=58-recovery-position
Whiteboard: r+
i915 platform: i915 features:
Attachments: channel was approved
channel was not yet approved
[PATCH 1/3] fd.o #34025: call ObserveChannels for channels waiting in CDOs when recovering
[PATCH 2/3] Regression test for passing unapproved channels to recovering observers
[PATCH 3/3] update autogenerated gtk-doc goo

Description Guillaume Desmottes 2011-02-08 02:29:21 UTC
MC doesn't call ObserveChannels() on an Observer having Recover if the channel has not be approved yet. See these 2 MC logs when restarting gnome-shell.
Comment 1 Guillaume Desmottes 2011-02-08 02:29:53 UTC
Created attachment 43089 [details]
channel was approved
Comment 2 Guillaume Desmottes 2011-02-08 02:30:12 UTC
Created attachment 43090 [details]
channel was not yet approved
Comment 3 Vivek Dasmohapatra 2011-02-08 04:20:49 UTC
On my list. Will get to it as soon as I can, probably tomorrow, certainly this week.
Comment 4 Simon McVittie 2011-05-19 05:44:59 UTC
I looked at this briefly and spotted the failure mode; patches on the way.
Comment 5 Simon McVittie 2011-05-19 05:45:44 UTC
Created attachment 46899 [details] [review]
[PATCH 1/3] fd.o #34025: call ObserveChannels for channels waiting  in CDOs when recovering
Comment 6 Simon McVittie 2011-05-19 05:46:10 UTC
Created attachment 46900 [details] [review]
[PATCH 2/3] Regression test for passing unapproved channels to  recovering observers

I've confirmed that this fails without patch 1/3.
Comment 7 Simon McVittie 2011-05-19 05:46:44 UTC
Created attachment 46901 [details] [review]
[PATCH 3/3] update autogenerated gtk-doc goo

Not really related to this bug, but it's been annoying me every time I rebuild MC...
Comment 8 Guillaume Desmottes 2011-05-19 08:19:30 UTC
Looks good
Comment 9 Simon McVittie 2011-05-19 08:45:15 UTC
Thanks, fixed in git for 5.8.0, 5.9.1.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.