~vanvugt/unity/fix-742544-alt-trunk

Viewing all changes in revision 1793.

  • Committer: Tarmac
  • Author(s): Andrea Azzarone
  • Date: 2011-12-20 08:35:42 UTC
  • mfrom: (1738.11.7 fix-875467-2)
  • Revision ID: tarmac-20111220083542-b41ps1wuff0fv3th
Use unity::IconLoader to load device launcher icon that will be used to show an OSD notification. If it's not able to load an icon it shows a OSD notification without an icon.

This merge propose comes from this one (https://code.launchpad.net/~andyrock/unity/fix-875467/+merge/79638). In that merge proposal I've created a mini (very very mini) icon loader implementation. Reading unity code I noticed that we already use two icon loader implementations (the dash icon loader and the launcher ones) so...

Because unity::IconLoader has already been tested and in a unit test we should not test an external module (osd notification library), i don't know if we need a unit test for this change. In the case let me know.. Fixes: https://bugs.launchpad.net/bugs/875467. Appoved by Tim Penhey, Marco Trevisan (Treviño).

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: