[browser] favicons are not persistently cached on disk

Bug #1340585 reported by Olivier Tilloy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Triaged
Wishlist
Unassigned
webbrowser-app
Fix Released
Medium
Olivier Tilloy
ubuntu-ui-toolkit (Ubuntu)
Triaged
Wishlist
Unassigned
webbrowser-app (Ubuntu)
Fix Released
Undecided
Olivier Tilloy

Bug Description

Since version 0.23+14.10.20140709-0ubuntu1 of webbrowser-app, favicon support was re-introduced (it had been initially removed when switching to oxide). As of that version, the bookmarks list on the new tab page displays favicons. However the favicons are not cached on disk, all we store is a reference to their URL (e.g. http://assets.ubuntu.com/sites/ubuntu/latest/u/img/favicon.ico). Once the icons have been fetched by the QML engine, they will be cached in memory during the lifetime of the application, but if the application is killed and restarted, the icons will be fetched from the network again, potentially incurring a cost on the data connection.

Favicons should be cached on disk to avoid re-downloading them every time the app is launched.

Related branches

Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Olivier Tilloy (osomon) wrote :

I added an ubuntu-ui-toolkit task to this bug as there was a mention a few weeks back of a generic and transparent mechanism in the UITK to implement a persistent image cache.

As far as I know, the feature was first implemented in unity8 (to persistently cache icons in the apps scope), with a plan to make it more generic and move it to the UITK in the future (see this thread for reference: https://lists.launchpad.net/ubuntu-phone/msg08632.html).

Olivier Tilloy (osomon)
Changed in webbrowser-app:
assignee: nobody → Olivier Tilloy (osomon)
Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: Triaged → Confirmed
status: Confirmed → In Progress
Revision history for this message
Zsombor Egri (zsombi) wrote :

As far as I know there were no plans to have such a cache in toolkit. However we would be glad if someone would contribute that to the toolkit.

Changed in ubuntu-ui-toolkit:
status: New → Triaged
importance: Undecided → Wishlist
summary: - favicons are not persistently cached on disk
+ [browser] favicons are not persistently cached on disk
Olivier Tilloy (osomon)
Changed in webbrowser-app (Ubuntu):
status: New → In Progress
assignee: nobody → Olivier Tilloy (osomon)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package webbrowser-app - 0.23+14.10.20141003-0ubuntu1

---------------
webbrowser-app (0.23+14.10.20141003-0ubuntu1) utopic; urgency=low

  [ CI bot ]
  * Resync trunk

  [ Alexandre Abreu ]
  * Add a simple mechanism (simpler than the one already in place) to
    override user agents for a given webapp. (LP: #1370975)

  [ Alberto Mardegan ]
  * Fix handling of host cookies

  [ Olivier Tilloy ]
  * Ship a PNG icon instead of an SVG, and install it under the app’s
    own data dir. (LP: #1365408)
  * Make an autopilot test more robust by ensuring that the drawer menu
    is dismissed but not clicked. (LP: #1365304)
  * Persistently cache favicons on disk to avoid re-downloading them
    across application runs. (LP: #1340585)
  * Ensure that at most 2 webviews are instantiated at all times, to
    reduce memory consumption. (LP: #1376418)
 -- Ubuntu daily release <email address hidden> Fri, 03 Oct 2014 14:42:27 +0000

Changed in webbrowser-app (Ubuntu):
status: In Progress → Fix Released
Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: In Progress → Fix Released
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → Wishlist
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.