~ken-vandine/signon-ui/arches

Viewing all changes in revision 86.

  • Committer: Tarmac
  • Author(s): Alberto Mardegan
  • Date: 2013-04-05 15:51:43 UTC
  • mfrom: (85.1.2 embed-crash)
  • Revision ID: tarmac-20130405155143-olc2x3kcvtnqvibf
Don't crash when re-embedding into the same window ID

This fixes a crash which I happened to reproduce while testing: X reuses the X 
window ID, so it's possible that two different instances of the XEmbed
container assume the same window ID (obviously not at the same time); however,
the QX11EmbedWidget gets destroyed as soon as the first container is closed,
and it was not getting removed from the pool.
.

Approved by Ken VanDine, PS Jenkins bot.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: