Bug 1388396: Use IconRequestBuilder.forActivityStream in AS UI. r=sebastian
MozReview-Commit-ID: jLVtXk0hJZ
--- a/mobile/android/base/java/org/mozilla/gecko/activitystream/homepanel/stream/StreamOverridablePageIconLayout.java
+++ b/mobile/android/base/java/org/mozilla/gecko/activitystream/homepanel/stream/StreamOverridablePageIconLayout.java
@@ -73,16 +73,17 @@ public class StreamOverridablePageIconLa
.centerCrop()
.into(imageView);
} else {
setUIMode(UIMode.FAVICON_IMAGE);
ongoingFaviconLoad = Icons.with(getContext())
.pageUrl(pageURL)
.skipNetwork()
+ .forActivityStream()
.build()
.execute(this);
}
}
@Override
public void onIconResponse(final IconResponse response) {
faviconView.updateImage(response);
--- a/mobile/android/base/java/org/mozilla/gecko/activitystream/homepanel/topsites/TopSitesCard.java
+++ b/mobile/android/base/java/org/mozilla/gecko/activitystream/homepanel/topsites/TopSitesCard.java
@@ -101,16 +101,17 @@ import java.util.concurrent.Future;
// Sometimes we get top sites without or with an empty URL - even though we do not allow
// this anywhere in our UI. However with 'sync' we are not in full control of the data.
// Whenever the URL is empty or null we just clear a potentially previously set icon.
faviconView.clearImage();
} else {
ongoingIconLoad = Icons.with(itemView.getContext())
.pageUrl(topSite.getUrl())
.skipNetwork()
+ .forActivityStream()
.build()
.execute(this);
}
final Drawable pinDrawable;
if (topSite.isPinned()) {
pinDrawable = DrawableUtil.tintDrawable(itemView.getContext(), R.drawable.as_pin, Color.WHITE);
} else {