Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(457)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java

Issue 956303003: Fix favicon padding in Site Settings on pre-L devices. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java
index 2158617ce380e80a860e01f2ec7d6401d7538615..17dab5a6af322d1b661145c2044708d5bc82d8cb 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreference.java
@@ -14,7 +14,6 @@ import android.preference.Preference;
import android.text.format.Formatter;
import android.view.View;
import android.widget.ImageView;
-import android.widget.LinearLayout;
import android.widget.TextView;
import org.chromium.base.annotations.SuppressFBWarnings;
@@ -46,12 +45,9 @@ class WebsitePreference extends Preference implements FaviconImageCallback {
// Metrics for favicon processing.
private static final int FAVICON_CORNER_RADIUS_DP = 2;
private static final int FAVICON_SIZE_DP = 16;
+ private static final int FAVICON_PADDING_DP = 4;
private static final int FAVICON_TEXT_SIZE_SP = 10;
private static final int FAVICON_BACKGROUND_COLOR = 0xff969696;
- // The minimum width of the preference icon parent field.
- private static final int FAVICON_PARENT_MINWIDTH_DP = 55;
- // The padding for the preference icon parent field.
- private static final int FAVICON_PARENT_PADDING_DP = 12;
WebsitePreference(Context context, Website site, String categoryFilter) {
super(context);
@@ -61,15 +57,11 @@ class WebsitePreference extends Preference implements FaviconImageCallback {
setWidgetLayoutResource(R.layout.website_features);
// To make sure the layout stays stable throughout, we assign a
- // transparent drawable of the same size as the favicon. This is so that
+ // transparent drawable as the icon initially. This is so that
// we can fetch the favicon in the background and not have to worry
// about the title appearing to jump (http://crbug.com/453626) when the
// favicon becomes available.
- ColorDrawable drawable = new ColorDrawable(Color.TRANSPARENT);
- int size = Math.round(FAVICON_SIZE_DP
- * getContext().getResources().getDisplayMetrics().density);
- drawable.setBounds(0, 0, size, size);
- setIcon(drawable);
+ setIcon(new ColorDrawable(Color.TRANSPARENT));
refresh();
}
@@ -146,8 +138,8 @@ class WebsitePreference extends Preference implements FaviconImageCallback {
TextView usageText = (TextView) view.findViewById(R.id.usage_text);
usageText.setVisibility(View.GONE);
- long totalUsage = mSite.getTotalUsage();
if (mFilter.showStorageSites(mCategoryFilter)) {
+ long totalUsage = mSite.getTotalUsage();
if (totalUsage > 0) {
usageText.setText(Formatter.formatShortFileSize(getContext(), totalUsage));
usageText.setTextSize(TEXT_SIZE_SP);
@@ -181,15 +173,9 @@ class WebsitePreference extends Preference implements FaviconImageCallback {
mFaviconFetched = true;
}
- ImageView icon = (ImageView) view.findViewById(android.R.id.icon);
- View parent = (View) icon.getParent();
- if (parent instanceof LinearLayout) {
- LinearLayout parentLayout = (LinearLayout) parent;
- int minWidth = Math.round(FAVICON_PARENT_MINWIDTH_DP * density);
- int padding = Math.round(FAVICON_PARENT_PADDING_DP * density);
- parentLayout.setMinimumWidth(minWidth);
- parentLayout.setPadding(padding, 0, padding, 0);
- }
+ int iconPadding = Math.round(FAVICON_PADDING_DP * density);
+ View iconView = view.findViewById(android.R.id.icon);
+ iconView.setPadding(iconPadding, iconPadding, iconPadding, iconPadding);
}
/**
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698