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

Proposed by Lars Karlitski
Status: Merged
Approved by: Ted Gould
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
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Approve
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.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
445. By Lars Karlitski

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 :

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 :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

Cool, thanks for running this correctly Jenkins :-/

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches