Merge lp:~aacid/unity8/make_try_no_cpu_usage into lp:unity8
Status: | Merged |
---|---|
Approved by: | Michael Zanetti |
Approved revision: | 1793 |
Merged at revision: | 1811 |
Proposed branch: | lp:~aacid/unity8/make_try_no_cpu_usage |
Merge into: | lp:unity8 |
Prerequisite: | lp:~aacid/unity8/oxide_regression_workaround |
Diff against target: |
20 lines (+2/-1) 1 file modified
tests/utils/modules/Unity/Test/UnityTestCase.qml (+2/-1) |
To merge this branch: | bzr merge lp:~aacid/unity8/make_try_no_cpu_usage |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Zanetti (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+260314@code.launchpad.net |
This proposal supersedes a proposal from 2015-05-27.
Commit message
Bind running to the proper object.visible
The parent of that rotationAnimation is not the rotatingRectangle even if you'd expect it to be
Fixes continous CPU usage on make tryXXX
Description of the change
* Are there any related MPs required for this MP to build/function as expected?
No
* Did you perform an exploratory manual test run of your code change and any related functionality?
Test change only
* Did you make sure that your branch does not contain spurious tags?
Yes
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
* If you changed the UI, has there been a design review?
N/A
* Did you have a look at the warnings when running tests?
No new warnings
Yes!
* Did you perform an exploratory manual test run of the code change and any related functionality?
yes
* Did CI run pass? If not, please explain why.
not yet, sure it will
* Did you make sure that the branch does not contain spurious tags?
yes