Merge lp://qastaging/~vanvugt/unity/fix-687567-unity3.0 into lp://qastaging/unity/3.0

Proposed by Daniel van Vugt
Status: Merged
Merged at revision: 1207
Proposed branch: lp://qastaging/~vanvugt/unity/fix-687567-unity3.0
Merge into: lp://qastaging/unity/3.0
Diff against target: 146 lines (+37/-66)
1 file modified
services/panel-service.c (+37/-66)
To merge this branch: bzr merge lp://qastaging/~vanvugt/unity/fix-687567-unity3.0
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+67816@code.qastaging.launchpad.net

Description of the change

Fix white "flashes" seen in the panel menus during scrubbing, or when they're unmapped (LP: #687567)

The problem, it seems, was that we were calling gdk/gtk/X11 functions in the middle of a GDK event filter. Removing these function calls and replacing them with a simpler, more optimized solution solves the problem.

I'm proposing this fix to unity 3.0 first because it was developed and tested on natty. I will forward port it to unity trunk (requires some conflict resolution) after this version has been approved.

To post a comment you must log in.
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

And before someone suggests the bug was a timing issue, it wasn't. I tried inserting delays in the affected code and it made no difference. Only removing the gdk/gtk/X calls fixed the problem.

Revision history for this message
Neil J. Patel (njpatel) wrote :

*hattips*

Very, very nice merge :) I couldn't compile old Unity on my system so couldn't test your branch but I did a quick port of your work to the latest unity (diff: http://paste.ubuntu.com/643341/) and it works perfectly!

If you want to merge-propose something for trunk, I'll compile and test this on my natty desktop this evening.

Revision history for this message
Neil J. Patel (njpatel) wrote :

Works perfectly, approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: