Merge lp:~davidc3/unity-scope-chromiumbookmarks/icons-n-dups into lp:unity-scope-chromiumbookmarks
Proposed by
David Callé
Status: | Merged |
---|---|
Approved by: | Paweł Stołowski |
Approved revision: | 30 |
Merged at revision: | 28 |
Proposed branch: | lp:~davidc3/unity-scope-chromiumbookmarks/icons-n-dups |
Merge into: | lp:unity-scope-chromiumbookmarks |
Diff against target: |
106 lines (+20/-19) 1 file modified
src/unity_chromiumbookmarks_daemon.py (+20/-19) |
To merge this branch: | bzr merge lp:~davidc3/unity-scope-chromiumbookmarks/icons-n-dups |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paweł Stołowski (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email:
|
Commit message
Fix icons and duplicated results
Description of the change
This branch:
- deduplicates results by ensuring that profiles are only added once to the parsing list.
- prevents an unrelated icon to be used for the next result.
- Cleans the result set creation
To post a comment you must log in.
PASSED: Continuous integration, rev:28 jenkins. qa.ubuntu. com/job/ unity-scope- chromiumbookmar ks-ci/3/ jenkins. qa.ubuntu. com/job/ unity-scope- chromiumbookmar ks-raring- amd64-ci/ 3
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ unity-scope- chromiumbookmar ks-ci/3/ rebuild
http://