Comment on attachment 8907334 [details] Bug 1399226 - Add high-res icons, image placeholders and bug fixes to Activity Stream. https://reviewboard.mozilla.org/r/179008/#review184102 ::: browser/extensions/activity-stream/data/content/tippytop/top_sites.json:39 (Diff revision 1) > "url": "https://www.facebook.com/", > "image_url": "facebook-com@2x.png" > }, > { > "title": "leboncoin", > "url": "http://www.leboncoin.fr/", Based on the website, I think this wants to be https://www.leboncoin.fr/. That shouldn't block this from landing, but probably worth filing an issues in github. ::: browser/extensions/activity-stream/lib/TelemetryFeed.jsm:206 (Diff revision 1) > perf: {load_trigger_type: "unexpected"} > }; > > + if (url) { > + session.page = url; > + } This is harmless, but I meant to remove it. I'll file a github issue for that.

Comment on attachment 8907334 [details] Bug 1399226 - Add high-res icons, image placeholders and bug fixes to Activity Stream. https://reviewboard.mozilla.org/r/179008/#review184106 Looks good, just a couple of nits that don't block landing it.

(In reply to Dan Mosedale (:dmose) from comment #2) > browser/extensions/activity-stream/data/content/tippytop/top_sites.json:39 > > "url": "http://www.leboncoin.fr/", > Based on the website, I think this wants to be https://www.leboncoin.fr/. We actually show the https as the top site, but for matching tippy top image, it doesn't care about protocol.

Pushed by edilee@gmail.com: https://hg.mozilla.org/integration/autoland/rev/82f171552905 Add high-res icons, image placeholders and bug fixes to Activity Stream. r=dmose