Merge lp:~aacid/unity8/make_try_no_cpu_usage into lp:unity8

Proposed by Albert Astals Cid
Status: Superseded
Proposed branch: lp:~aacid/unity8/make_try_no_cpu_usage
Merge into: lp:unity8
Diff against target: 30 lines (+3/-2)
2 files modified
tests/qmltests/CMakeLists.txt (+1/-1)
tests/utils/modules/Unity/Test/UnityTestCase.qml (+2/-1)
To merge this branch: bzr merge lp:~aacid/unity8/make_try_no_cpu_usage
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michael Zanetti (community) Approve
Review via email: mp+260288@code.launchpad.net

This proposal has been superseded by a proposal from 2015-05-27.

Commit message

Anchor 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

To post a comment you must log in.
Revision history for this message
Michael Zanetti (mzanetti) wrote :

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

review: Approve
1793. By Albert Astals Cid

Merge oxide_regression_workaround

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/qmltests/CMakeLists.txt'
2--- tests/qmltests/CMakeLists.txt 2015-05-18 23:04:19 +0000
3+++ tests/qmltests/CMakeLists.txt 2015-05-27 14:21:44 +0000
4@@ -93,4 +93,4 @@
5 add_qml_test(Stages WindowMoveResizeArea)
6 add_qml_test(Stages Splash)
7 add_qml_test(Tutorial Tutorial)
8-add_qml_test(Wizard Wizard)
9+add_qml_test(Wizard Wizard ENVIRONMENT "OXIDE_NO_SANDBOX=1")
10
11=== modified file 'tests/utils/modules/Unity/Test/UnityTestCase.qml'
12--- tests/utils/modules/Unity/Test/UnityTestCase.qml 2015-04-17 18:31:12 +0000
13+++ tests/utils/modules/Unity/Test/UnityTestCase.qml 2015-05-27 14:21:44 +0000
14@@ -26,6 +26,7 @@
15 // This is needed for waitForRendering calls to return
16 // if the watched element already got rendered
17 Rectangle {
18+ id: rotatingRectangle
19 width: units.gu(1)
20 height: width
21 parent: testCase.parent
22@@ -35,7 +36,7 @@
23 visible: testCase.running
24
25 RotationAnimation on rotation {
26- running: parent.visible
27+ running: rotatingRectangle.visible
28 from: 0
29 to: 360
30 loops: Animation.Infinite

Subscribers

People subscribed via source and target branches