Merge lp://qastaging/~brandontschaefer/unity/lp.1161726-fix into lp://qastaging/unity
Proposed by
Brandon Schaefer
Status: | Merged |
---|---|
Approved by: | Stephen M. Webb |
Approved revision: | no longer in the source branch. |
Merged at revision: | 3265 |
Proposed branch: | lp://qastaging/~brandontschaefer/unity/lp.1161726-fix |
Merge into: | lp://qastaging/unity |
Diff against target: |
30 lines (+10/-1) 2 files modified
launcher/EdgeBarrierController.cpp (+1/-1) tests/test_edge_barrier_controller.cpp (+9/-0) |
To merge this branch: | bzr merge lp://qastaging/~brandontschaefer/unity/lp.1161726-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Stephen M. Webb (community) | Approve | ||
MC Return (community) | Approve | ||
Review via email: mp+156451@code.qastaging.launchpad.net |
Commit message
Release the barrier if there are no launcher/
Description of the change
=== Problem ===
If there is no subscriber for the monitor it was tell the EdgeBarrier to check if the velocity was large enough to break the barrier, even though there is no subscriber.
=== Fix ===
Change the default value from IGNORE to RELEASE_BARRIER. So if there are not enough subscribers then we just release the barrier.
=== Test ===
Unit test
To post a comment you must log in.
+1 from me.