Merge lp:~compiz-team/compiz/compiz.fix_1053280 into lp:compiz/0.9.8
Status: | Superseded |
---|---|
Proposed branch: | lp:~compiz-team/compiz/compiz.fix_1053280 |
Merge into: | lp:compiz/0.9.8 |
Diff against target: |
1115 lines (+859/-69) 10 files modified
include/core/action.h (+12/-1) include/core/option.h (+4/-2) src/action.cpp (+24/-0) src/event.cpp (+149/-62) src/eventmanagement.h (+80/-0) src/option.cpp (+18/-0) src/option/tests/option.cpp (+31/-0) src/privateaction.h (+25/-1) src/privatescreen/tests/test-privatescreen.cpp (+513/-1) src/screen.cpp (+3/-2) |
To merge this branch: | bzr merge lp:~compiz-team/compiz/compiz.fix_1053280 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Needs Resubmitting | ||
jenkins (community) | continuous-integration | Approve | |
Sam Spilsbury | Needs Resubmitting | ||
Review via email: mp+125984@code.launchpad.net |
This proposal supersedes a proposal from 2012-09-22.
This proposal has been superseded by a proposal from 2012-09-28.
Description of the change
Don't trigger actions unless they are actually marked active. It is possible that we may have a passive button grab for other reasons than actions being bound, so actions which haven't been added should be ignored.
Tests added for refactored code:
Start 107: privatescreen_
107/548 Test #107: privatescreen_
Start 108: privatescreen_
108/548 Test #108: privatescreen_
Start 109: privatescreen_
109/548 Test #109: privatescreen_
Start 110: privatescreen_
110/548 Test #110: privatescreen_
Start 111: privatescreen_
111/548 Test #111: privatescreen_
Start 112: privatescreen_
112/548 Test #112: privatescreen_
Start 113: privatescreen_
113/548 Test #113: privatescreen_
Start 114: privatescreen_
114/548 Test #114: privatescreen_
Start 115: privatescreen_
115/548 Test #115: privatescreen_
Start 116: privatescreen_
116/548 Test #116: privatescreen_
Start 117: privatescreen_
117/548 Test #117: privatescreen_
Start 118: privatescreen_
118/548 Test #118: privatescreen_
Start 119: privatescreen_
119/548 Test #119: privatescreen_
Start 120: privatescreen_
120/548 Test #120: privatescreen_
Start 121: privatescreen_
121/548 Test #121: privatescreen_
Unmerged revisions
- 3399. By Daniel van Vugt
-
Fix build error/warning: Remove unused function.
- 3398. By Sam Spilsbury
-
Merge lp:compiz
- 3397. By Sam Spilsbury
-
Forward declare correctly
- 3396. By Sam Spilsbury
-
Cleanup.
Unindent namespaces, simplify interface, add namespace alias where
appropriate. - 3395. By Sam Spilsbury
-
Constify the API a bit
- 3394. By Sam Spilsbury
-
Merge lp:compiz and handle some BCI stuff
- 3393. By Sam Spilsbury
-
Move check to the right place
- 3392. By Sam Spilsbury
-
Merge lp:compiz
- 3391. By Sam Spilsbury
-
Add missing file
- 3390. By Sam Spilsbury
-
Don't trigger actions unless they're actually added
FAILED: Continuous integration, rev:3390 jenkins. qa.ubuntu. com/job/ compiz- ci/95/ jenkins. qa.ubuntu. com/job/ compiz- ci/./build= pbuilder, distribution= quantal, flavor= amd64/95/ console
http://
Executed test runs:
FAILURE: http://