Code review comment for lp:~jassmith/unity/unity.fix-monitor-tracking

Revision history for this message
Tim Penhey (thumper) wrote :

<thumper> jason: ok... I'm going to ask the question you love to hate
<jason> no there is no bug
<thumper> jason: what is the behaviour before the commit, and how do I know it is fixed?
<jason> oh that question
<jason> before:
<jason> sometimes when you started an application unity would think it was on monitor 0 instead of monitor 1 because of a race condition
<jason> after:
<jason> we check, thus eliminating the race condition
<thumper> and to reproduce the behaviour?
<thomi> possible switcher bug that should be easy to fix: https://bugs.launchpad.net/unity/+bug/938353
<jason> thumper, restart unity enough times to eventually trigger the race condition

review: Approve

« Back to merge proposal