Bug 975198 - Don't use default favicon as placeholder in PanelItemView (r=margaret)
authorLucas Rocha <lucasr@mozilla.com>
Tue, 25 Feb 2014 13:32:59 +0000
changeset 170471 59b615846cf273277efc9434d5ba9705187d352c
parent 170470 fad94053c6b0193cfc46c6e25970616dd75d7b28
child 170472 e29ea34afb6c41062f3201ac82a2199d5dd5d214
push id26288
push userryanvm@gmail.com
push dateTue, 25 Feb 2014 20:20:43 +0000
treeherdermozilla-central@22650589a724 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmargaret
bugs975198
milestone30.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 975198 - Don't use default favicon as placeholder in PanelItemView (r=margaret)
mobile/android/base/home/PanelItemView.java
--- a/mobile/android/base/home/PanelItemView.java
+++ b/mobile/android/base/home/PanelItemView.java
@@ -61,17 +61,16 @@ class PanelItemView extends LinearLayout
 
         // Only try to load the image if the item has define image URL
         final boolean hasImageUrl = !TextUtils.isEmpty(imageUrl);
         mImage.setVisibility(hasImageUrl ? View.VISIBLE : View.GONE);
 
         if (hasImageUrl) {
             Picasso.with(getContext())
                    .load(imageUrl)
-                   .error(R.drawable.favicon)
                    .into(mImage);
         }
     }
 
     private static class ArticleItemView extends PanelItemView {
         private ArticleItemView(Context context) {
             super(context, R.layout.panel_article_item);
             setOrientation(LinearLayout.HORIZONTAL);