Merge lp:~midori/midori/thoughThaltNotCancelYourIcons into lp:midori

Proposed by Cris Dywan
Status: Merged
Approved by: Paweł Forysiuk
Approved revision: 6641
Merged at revision: 6641
Proposed branch: lp:~midori/midori/thoughThaltNotCancelYourIcons
Merge into: lp:midori
Diff against target: 26 lines (+3/-2)
2 files modified
katze/midori-uri.vala (+2/-1)
midori/webkitgtk-3.0.vapi (+1/-1)
To merge this branch: bzr merge lp:~midori/midori/thoughThaltNotCancelYourIcons
Reviewer Review Type Date Requested Status
Paweł Forysiuk Approve
Review via email: mp+213946@code.launchpad.net

Commit message

We must not pass a Cancellable to FaviconDatabase.get_favicon_pixbuf

To post a comment you must log in.
Revision history for this message
Paweł Forysiuk (tuxator) wrote :

Seems to fix the crasher here.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'katze/midori-uri.vala'
2--- katze/midori-uri.vala 2014-03-03 01:17:05 +0000
3+++ katze/midori-uri.vala 2014-04-02 23:59:21 +0000
4@@ -236,7 +236,8 @@
5 var pixbuf = Gdk.pixbuf_get_from_surface (image, 0, 0, image.get_width (), image.get_height ());
6 #else
7 var database = WebKit.get_favicon_database ();
8- var pixbuf = yield database.get_favicon_pixbuf (uri, 0, 0, cancellable);
9+ // We must not pass a Cancellable due to a crasher bug
10+ var pixbuf = yield database.get_favicon_pixbuf (uri, 0, 0, null);
11 #endif
12 return pixbuf as GLib.Icon;
13 }
14
15=== modified file 'midori/webkitgtk-3.0.vapi'
16--- midori/webkitgtk-3.0.vapi 2014-03-03 01:17:05 +0000
17+++ midori/webkitgtk-3.0.vapi 2014-04-02 23:59:21 +0000
18@@ -6,7 +6,7 @@
19 public class FaviconDatabase : GLib.Object {
20 public signal void icon_loaded (string frame_uri);
21 public Gdk.Pixbuf? try_get_favicon_pixbuf (string page_uri, uint width, uint height);
22- public async Gdk.Pixbuf? get_favicon_pixbuf (string page_uri, uint width, uint height, GLib.Cancellable cancellable) throws GLib.Error;
23+ public async Gdk.Pixbuf? get_favicon_pixbuf (string page_uri, uint width, uint height, GLib.Cancellable? cancellable) throws GLib.Error;
24 public void set_path (string? path);
25 public void clear ();
26 }

Subscribers

People subscribed via source and target branches

to all changes: