Messaging Menu

Merge lp://staging/~larsu/indicator-messages/update-root-action into lp://staging/indicator-messages/15.04

update-root-action
Merge into trunk.15.04

Proposed by Lars Karlitski on 2015-05-04

Status:	Merged
Approved by:	Ted Gould on 2015-05-05
Approved revision:	445
Merged at revision:	446
Proposed branch:	lp://staging/~larsu/indicator-messages/update-root-action
Merge into:	lp://staging/indicator-messages/15.04
Diff against target:	50 lines (+9/-10) 1 file modified src/im-application-list.c (+9/-10)
To merge this branch:	bzr merge lp://staging/~larsu/indicator-messages/update-root-action
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Ted Gould (community)		2015-05-04	Approve on 2015-05-05
PS Jenkins bot (community)	continuous-integration		Approve on 2015-05-04
Review via email: mp+258170@code.staging.launchpad.net

Commit message

application list: update root action unconditionally

Description of the change

application list: update root action unconditionally

Until now, the _update_root_action() was only called when the draws_attention was set. That function now also sets the sensitive state of some other actions, which might change independently of the attention flag.

Simply call the function unconditionally. It's not doing a lot of work anyway and it only gets called when something in the source or message lists changes.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-05-04:

FAILED: Continuous integration, rev:445
http://jenkins.qa.ubuntu.com/job/indicator-messages-ci/149/
Executed test runs:
FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-messages-vivid-amd64-ci/11/console
FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-messages-vivid-armhf-ci/11/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/indicator-messages-ci/149/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~larsu/indicator-messages/update-root-action updated on 2015-05-04

445. By Lars Karlitski on 2015-05-04

application list: update root action unconditionally

Until now, the _update_root_action() was only called when the
draws_attention was set. That function now also sets the sensitive state
of some other actions, which might change independently of the attention
flag.

Simply call the function unconditionally. It's not doing a lot of work
anyway and it only gets called when something in the source or message
lists changes.

Revision history for this message

Lars Karlitski (larsu) wrote on 2015-05-04:

I accidentally removed the calls to update_attention() as well, which was fortunately caught by the test.

Amended the commit to do the right thing.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-05-04:

FAILED: Continuous integration, rev:445
http://jenkins.qa.ubuntu.com/job/indicator-messages-ci/150/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/indicator-messages-vivid-amd64-ci/12/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-messages-vivid-armhf-ci/12
        deb: http://jenkins.qa.ubuntu.com/job/indicator-messages-vivid-armhf-ci/12/artifact/work/output/*zip*/output.zip