x11 cleanup must still be done in the UI thread

[totaam]

Issue migrated from trac ticket # 582

component: server | priority: minor | resolution: fixed

2014-05-28 04:45:16: totaam created the issue

---

Which is more difficult when we exit on a signal..

[[BR]]

Found this stacktrace in some automated test samples:

Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/gtk_common/gobject_util.py", line 30, in do_get_property
    return getattr(self, getter)(pspec.name)
  File "/usr/lib64/python2.7/site-packages/xpra/x11/gtk_x11/composite.py", line 205, in do_get_property_contents_handle
    trap.swallow_synced(set_pixmap)
  File "/usr/lib64/python2.7/site-packages/xpra/gtk_common/error.py", line 136, in swallow_synced
    self.call_synced(fun, *args, **kwargs)
  File "/usr/lib64/python2.7/site-packages/xpra/gtk_common/error.py", line 119, in call_synced
    return self._call(True, fun, args, kwargs)
  File "/usr/lib64/python2.7/site-packages/xpra/gtk_common/error.py", line 111, in _call
    raise e
AttributeError: 'NoneType' object has no attribute 'xid'

Which can only occur if self._window is set to None by destroy concurrently with a call to do_get_property_contents_handle...

[totaam]

2014-07-10 22:27:17: totaam changed priority from major to minor

[totaam]

2014-07-10 22:27:17: totaam changed status from new to assigned

[totaam]

2014-07-10 22:27:17: totaam edited the issue description

[totaam]

2014-07-10 22:27:17: totaam changed owner from antoine to totaam

[totaam]

2014-07-10 22:27:17: totaam commented

---

This particular error will no longer be shown as of r6888.

But the underlying issue remains... I am lowering the priority as this only affects exit-on-signal, which makes it less important. See also #626

[totaam]

2015-01-18 10:55:18: totaam commented

---

See also #626

[totaam]

2015-04-29 09:19:46: antoine changed status from assigned to closed

[totaam]

2015-04-29 09:19:46: antoine set resolution to fixed

[totaam]

2015-04-29 09:19:46: antoine commented

---

A similar problem was solved in r9190, I think that r9194 does the same thing for the regular server process. (not needed for the proxy server code)

Closing, will re-open if this breaks anything.

[totaam]

2015-05-12 19:02:16: antoine commented

---

Note: as per #626#comment:9, we don't cleanup from the UI thread when we exit from the signal handler... it seems we just can't have everything!