Dash previews cache cover-art images until unity restarts
Bug #1065427 reported by
Nick Dedekind
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Unity |
Triaged
|
Medium
|
Unassigned | ||
unity (Ubuntu) |
Triaged
|
Medium
|
Unassigned |
Bug Description
The dash preview art-work is cached by the icon loader until unity restarts. So the more you preview, the more memory is taken. This applies to all non-generated images, e.g. App preview screenshots, music album covers and file previews where an icon is shown.
Desired solution is to cache only while the dash previews are open to help smooth navigation.
Related branches
lp://staging/~nick-dedekind/unity/multi-instance-icon-loader
On hold
for merging
into
lp://staging/unity
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Michal Hruby (community): Needs Fixing
- Nick Dedekind (community): Needs Fixing
-
Diff: 2350 lines (+1138/-850)10 files modifieddash/ResultRendererTile.cpp (+2/-2)
dash/StandaloneDash.cpp (+3/-4)
unity-shared/CMakeLists.txt (+1/-0)
unity-shared/CoverArt.cpp (+14/-9)
unity-shared/CoverArt.h (+4/-2)
unity-shared/IconLoader.cpp (+112/-828)
unity-shared/IconLoader.h (+16/-5)
unity-shared/IconLoaderImpl.h (+119/-0)
unity-shared/IconLoaderTask.cpp (+756/-0)
unity-shared/IconLoaderTask.h (+111/-0)
description: | updated |
Changed in unity: | |
importance: | Undecided → Medium |
Changed in unity: | |
status: | New → Triaged |
Changed in unity (Ubuntu): | |
status: | New → In Progress |
Changed in unity: | |
status: | Triaged → In Progress |
assignee: | nobody → Nick Dedekind (nick-dedekind) |
milestone: | none → 7.0.0 |
Changed in unity: | |
milestone: | 7.0.0 → 7.0.1 |
Changed in unity: | |
milestone: | 7.0.1 → 7.3.1 |
Changed in unity: | |
status: | In Progress → Triaged |
Changed in unity (Ubuntu): | |
status: | In Progress → Confirmed |
Changed in unity: | |
milestone: | 7.3.1 → 7.3.2 |
Changed in unity: | |
milestone: | 7.3.2 → 7.3.3 |
Changed in unity (Ubuntu): | |
status: | Confirmed → Triaged |
importance: | Undecided → Medium |
Changed in unity: | |
assignee: | Nick Dedekind (nick-dedekind) → nobody |
milestone: | 7.3.3 → none |
tags: | added: backlog |
To post a comment you must log in.