Dash - Online preview content should be speculatively cached
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ayatana Design |
Fix Committed
|
High
|
John Lea | ||
Unity |
Triaged
|
Medium
|
Unassigned | ||
unity (Ubuntu) |
Triaged
|
Medium
|
Unassigned |
Bug Description
Reduce the incidence of this happening by speculatively pre-cacheing the preview data.
Note the pre-caching is very important to reduce the incidence of the user seeing the loading state. All major browsers currently speculatively pre-cache content, so if we are match user's expectations regarding speed of content display (shaped by using web browsers) this is something we really need to do.
-------
Desired solution:
1. When the user opens a preview, results subsequent to the preview should start pre-cacheing.
2. Once a user has previewed one content item underneath a category header, as long as the Dash remains open we continue to speculatively cache the preview content for the other items under the category header (even when the user returns to the results view). The pre-caching stops when a) the user closes the Dash b) the user previews the a result from underneath another category header. In this second case the Dash then starts speculatively pre-cacheing the content from underneath this new category header.
Related branches
- Brandon Schaefer (community): Needs Fixing
-
Diff: 1176 lines (+675/-75)22 files modifiedUnityCore/CMakeLists.txt (+2/-0)
UnityCore/HomeLens.cpp (+2/-2)
UnityCore/HomeLens.h (+1/-1)
UnityCore/Lens.cpp (+25/-25)
UnityCore/Lens.h (+3/-1)
UnityCore/PreviewPreCacher.cpp (+270/-0)
UnityCore/PreviewPreCacher.h (+94/-0)
dash/DashView.cpp (+8/-3)
dash/LensView.cpp (+15/-13)
dash/LensView.h (+7/-3)
dash/previews/ApplicationPreview.cpp (+3/-3)
dash/previews/GenericPreview.cpp (+2/-2)
dash/previews/MoviePreview.cpp (+2/-2)
dash/previews/MusicPreview.cpp (+2/-2)
dash/previews/PreviewContainer.cpp (+1/-1)
dash/previews/PreviewInfoHintWidget.cpp (+1/-1)
dash/previews/SocialPreview.cpp (+3/-3)
dash/previews/SocialPreviewComments.cpp (+1/-1)
dash/previews/SocialPreviewContent.cpp (+4/-4)
tests/CMakeLists.txt (+1/-0)
tests/test_lens.cpp (+4/-8)
tests/test_previews_precacher.cpp (+224/-0)
tags: | added: udp |
Changed in ayatana-design: | |
assignee: | nobody → John Lea (johnlea) |
importance: | Undecided → High |
Changed in unity: | |
importance: | Undecided → High |
Changed in unity (Ubuntu): | |
importance: | Undecided → High |
tags: | added: dashm |
Changed in ayatana-design: | |
status: | New → Triaged |
Changed in unity: | |
status: | New → Triaged |
Changed in unity (Ubuntu): | |
status: | New → Triaged |
Changed in ayatana-design: | |
status: | Triaged → Fix Committed |
description: | updated |
Changed in unity: | |
assignee: | nobody → Nick Dedekind (nick-dedekind) |
status: | Triaged → In Progress |
Changed in unity: | |
milestone: | none → 7.0.0 |
Changed in unity (Ubuntu): | |
assignee: | nobody → Nick Dedekind (nick-dedekind) |
milestone: | none → ubuntu-13.04-month-3 |
status: | Triaged → In Progress |
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 |
Changed in unity: | |
importance: | High → Medium |
Changed in unity (Ubuntu): | |
importance: | High → Medium |
Changed in unity: | |
assignee: | Nick Dedekind (nick-dedekind) → nobody |
Changed in unity (Ubuntu): | |
assignee: | Nick Dedekind (nick-dedekind) → nobody |
Changed in unity: | |
milestone: | 7.3.3 → none |
Changed in unity (Ubuntu): | |
milestone: | ubuntu-13.04-month-3 → none |
tags: | added: backlog |
tags: | added: rls-w-incoming |
tags: |
added: rls-x-incoming removed: rls-w-incoming |