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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java

Issue 2912083003: Replace assets for avatar placeholder on Android (Closed)
Patch Set: Use vector asset for account avatar picture Created 3 years, 7 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
Index: chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java b/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java
index 86ec71574eaaac8a34e3deb1a882e82704c2635e..766b791c00352bc19d79bfa7b178fb0cf1214f26 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/password_manager/AccountChooserDialog.java
@@ -10,6 +10,7 @@ import android.content.DialogInterface;
import android.content.res.Resources;
import android.graphics.Bitmap;
import android.support.v7.app.AlertDialog;
+import android.support.v7.content.res.AppCompatResources;
import android.text.SpannableString;
import android.text.Spanned;
import android.text.TextUtils;
@@ -123,7 +124,8 @@ public class AccountChooserDialog
if (avatar != null) {
avatarView.setImageBitmap(avatar);
} else {
- avatarView.setImageResource(R.drawable.account_management_no_picture);
+ avatarView.setImageDrawable(AppCompatResources.getDrawable(
+ getContext(), R.drawable.logo_avatar_anonymous));
}
TextView mainNameView = (TextView) convertView.findViewById(R.id.main_name);
@@ -271,7 +273,7 @@ public class AccountChooserDialog
if (mIsDestroyed) return;
assert index >= 0 && index < mCredentials.length;
assert mCredentials[index] != null;
- avatarBitmap = AccountManagementFragment.makeRoundUserPicture(avatarBitmap);
+ avatarBitmap = AccountManagementFragment.makeRoundUserPicture(mContext, avatarBitmap);
mCredentials[index].setBitmap(avatarBitmap);
ListView view = mDialog.getListView();
if (index >= view.getFirstVisiblePosition() && index <= view.getLastVisiblePosition()) {

Powered by Google App Engine
This is Rietveld 408576698