Merge lp:~smspillaz/unity/unity.fix_835259 into lp:unity

Proposed by Sam Spilsbury
Status: Merged
Approved by: Sam Spilsbury
Approved revision: no longer in the source branch.
Merged at revision: 1672
Proposed branch: lp:~smspillaz/unity/unity.fix_835259
Merge into: lp:unity
Diff against target: 17 lines (+5/-2)
1 file modified
plugins/unityshell/src/unityshell.cpp (+5/-2)
To merge this branch: bzr merge lp:~smspillaz/unity/unity.fix_835259
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+77556@code.launchpad.net

Description of the change

Fix bug 835259

There's a race condition that can happen here where compiz would have already destroyed the window before bamf knows about the window being destroyed, so it will still be in bamf_view_get_children

To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/unityshell/src/unityshell.cpp'
--- plugins/unityshell/src/unityshell.cpp 2011-09-29 07:45:55 +0000
+++ plugins/unityshell/src/unityshell.cpp 2011-09-29 15:22:14 +0000
@@ -634,9 +634,12 @@
634 for (LayoutWindow::Ptr target : targets)634 for (LayoutWindow::Ptr target : targets)
635 {635 {
636 CompWindow* window = screen->findWindow(target->xid);636 CompWindow* window = screen->findWindow(target->xid);
637 UnityWindow *unity_window = UnityWindow::get (window);637 if (window)
638 {
639 UnityWindow *unity_window = UnityWindow::get (window);
638640
639 unity_window->paintThumbnail (target->result, target->alpha);641 unity_window->paintThumbnail (target->result, target->alpha);
642 }
640 }643 }
641 }644 }
642645