Bug 869634 - Update default favicon style. r=bnicholson

This commit is contained in:
Margaret Leibovic 2013-05-09 09:51:53 -04:00
Родитель 8ddd13e205
Коммит f70fac5938
4 изменённых файлов: 6 добавлений и 6 удалений

Двоичный файл не отображается.

До

Ширина:  |  Высота:  |  Размер: 1.8 KiB

После

Ширина:  |  Высота:  |  Размер: 1.3 KiB

Двоичный файл не отображается.

До

Ширина:  |  Высота:  |  Размер: 1.6 KiB

После

Ширина:  |  Высота:  |  Размер: 840 B

Двоичный файл не отображается.

До

Ширина:  |  Высота:  |  Размер: 2.1 KiB

После

Ширина:  |  Высота:  |  Размер: 1.8 KiB

Просмотреть файл

@ -34,16 +34,16 @@ public class FaviconView extends ImageView {
*/ */
public void updateImage(Bitmap bitmap, String key) { public void updateImage(Bitmap bitmap, String key) {
if (bitmap == null) { if (bitmap == null) {
// Call setImageDrawable directly to avoid creating a useless BitmapDrawable. // If the bitmap is null, show the default favicon.
setImageDrawable(null); setImageResource(R.drawable.favicon);
// If the bitmap is null, show a blank background. // The default favicon image is hi-res, so we should hide the background.
setBackgroundResource(R.drawable.favicon_bg); setBackgroundResource(0);
} else if (Favicons.getInstance().isLargeFavicon(bitmap)) { } else if (Favicons.getInstance().isLargeFavicon(bitmap)) {
super.setImageBitmap(bitmap); setImageBitmap(bitmap);
// If the icon is large, hide the background. // If the icon is large, hide the background.
setBackgroundResource(0); setBackgroundResource(0);
} else { } else {
super.setImageBitmap(bitmap); setImageBitmap(bitmap);
// Otherwise show a dominant color background. // Otherwise show a dominant color background.
int color = Favicons.getInstance().getFaviconColor(bitmap, key); int color = Favicons.getInstance().getFaviconColor(bitmap, key);
color = Color.argb(70, Color.red(color), Color.green(color), Color.blue(color)); color = Color.argb(70, Color.red(color), Color.green(color), Color.blue(color));