Bug 1397888: Remove pin icon from top sites title. r=liuche
authorMichael Comella <michael.l.comella@gmail.com>
Wed, 13 Sep 2017 16:58:39 -0700
changeset 432965 5289d258b56541f4cb5ddcf4496b899c23b1ce38
parent 432964 a54ba2e89fe093f416814fe5e261d2d9957261bb
child 432966 adab05485ec5f7966ef626981374f10ce54295f3
push id1567
push userjlorenzo@mozilla.com
push dateThu, 02 Nov 2017 12:36:05 +0000
treeherdermozilla-release@e512c14a0406 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersliuche
bugs1397888
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1397888: Remove pin icon from top sites title. r=liuche MozReview-Commit-ID: GTmfUzvD0k2
mobile/android/base/java/org/mozilla/gecko/activitystream/homepanel/topsites/TopSitesCard.java
--- 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
@@ -106,24 +106,16 @@ import java.util.concurrent.Future;
             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 {
-            pinDrawable = null;
-        }
-        TextViewCompat.setCompoundDrawablesRelativeWithIntrinsicBounds(title, pinDrawable, null, null, null);
-
         setTopSiteTitle(topSite);
     }
 
     private void setTopSiteTitle(final TopSite topSite) {
         URI topSiteURI = null; // not final so we can use in the Exception case.
         boolean isInvalidURI = false;
         try {
             topSiteURI = new URI(topSite.getUrl());