[Pkg-mozext-commits] [firegestures] 01/05: fix #107: error in mozlAsyncFavicons.getFaviconURLForPage

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository firegestures.

commit f21048ae80fe53f1407a6cfc9db59742b0caba21
Author: Gomita <gomita at xuldev.org>
Date:   Sat Oct 31 11:26:41 2015 +0900

    fix #107: error in mozlAsyncFavicons.getFaviconURLForPage
---
 chrome/content/firegestures/browser.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/chrome/content/firegestures/browser.js b/chrome/content/firegestures/browser.js
index d0c2f45..82bfde2 100644
--- a/chrome/content/firegestures/browser.js
+++ b/chrome/content/firegestures/browser.js
@@ -995,7 +995,8 @@ var FireGestures = {
 						menuitem.className = "unified-nav-current";
 					}
 					else {
-						PlacesUtils.favicons.getFaviconURLForPage(entry.URI, function(aURI) {
+						var entryURI = this.isRemote ? makeURI(entry.URI.spec) : entry.URI;
+						PlacesUtils.favicons.getFaviconURLForPage(entryURI, function(aURI) {
 							if (!aURI)
 								return;
 							let iconURL = PlacesUtils.favicons.getFaviconLinkForIcon(aURI).spec;

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firegestures.git