Merge lp:~submarine/unity-scope-firefoxbookmarks/ff23-thumbnails into lp:unity-scope-firefoxbookmarks

Proposed by David Callé
Status: Merged
Approved by: David Callé
Approved revision: 31
Merged at revision: 33
Proposed branch: lp:~submarine/unity-scope-firefoxbookmarks/ff23-thumbnails
Merge into: lp:unity-scope-firefoxbookmarks
Diff against target: 11 lines (+1/-0)
1 file modified
src/unity_firefoxbookmarks_daemon.py (+1/-0)
To merge this branch: bzr merge lp:~submarine/unity-scope-firefoxbookmarks/ff23-thumbnails
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Guillermo Gonzalez Approve
Review via email: mp+173609@code.launchpad.net

Commit message

Fix thumbnails fetching with FF23

Description of the change

Change thumbs search location (FF23)

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Guillermo Gonzalez (verterok) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/unity_firefoxbookmarks_daemon.py'
--- src/unity_firefoxbookmarks_daemon.py 2013-06-07 14:15:48 +0000
+++ src/unity_firefoxbookmarks_daemon.py 2013-07-08 22:01:27 +0000
@@ -105,6 +105,7 @@
105 if not bookmark[1].find("place:") == -1:105 if not bookmark[1].find("place:") == -1:
106 continue106 continue
107 path = bookmark[4].replace('places.sqlite', 'thumbnails/')107 path = bookmark[4].replace('places.sqlite', 'thumbnails/')
108 path = path.replace('.mozilla/', '.cache/mozilla/')
108 icon = '%s%s.png' % (path, hashlib.md5(bookmark[1].encode()).hexdigest())109 icon = '%s%s.png' % (path, hashlib.md5(bookmark[1].encode()).hexdigest())
109 if not os.path.exists(icon):110 if not os.path.exists(icon):
110 icon = None111 icon = None

Subscribers

People subscribed via source and target branches

to all changes: