Fix issue with "me" contact photo never showing up; show gray guy for default image
Change-Id: I9b02c17fc389737cb63831b5075610f39f510aa7
diff --git a/src/com/android/mail/photomanager/ContactPhotoManager.java b/src/com/android/mail/photomanager/ContactPhotoManager.java
index 323d1d3..2733023 100644
--- a/src/com/android/mail/photomanager/ContactPhotoManager.java
+++ b/src/com/android/mail/photomanager/ContactPhotoManager.java
@@ -361,7 +361,7 @@
@Override
public void loadThumbnail(DividedImageCanvas view, String name, String emailAddress,
DefaultImageProvider defaultProvider) {
- if (TextUtils.isEmpty(name)) {
+ if (TextUtils.isEmpty(emailAddress)) {
// No photo is needed
defaultProvider.applyDefaultImage(name, emailAddress, view, -1);
mPendingRequests.remove(view);
diff --git a/src/com/android/mail/photomanager/LetterTileProvider.java b/src/com/android/mail/photomanager/LetterTileProvider.java
index 2e1840f..d7f41a4 100644
--- a/src/com/android/mail/photomanager/LetterTileProvider.java
+++ b/src/com/android/mail/photomanager/LetterTileProvider.java
@@ -61,10 +61,13 @@
public void applyDefaultImage(String displayName, String address, DividedImageCanvas view,
int extent) {
Bitmap bitmap;
- final String display = !TextUtils.isEmpty(displayName) ? displayName : address;
- final char firstChar = Character.toUpperCase(display.charAt(0));
+ //final String display = !TextUtils.isEmpty(displayName) ? displayName : address;
+ //final char firstChar = Character.toUpperCase(display.charAt(0));
// If its a valid ascii character...
- if (Character.isLetter(firstChar)) {
+ // TODO: don't show tiles until we have a visual design
+ if (false) {
+ final String display = !TextUtils.isEmpty(displayName) ? displayName : address;
+ final char firstChar = Character.toUpperCase(display.charAt(0));
final String first = firstChar+"";
bitmap = mTileBitmapCache.get(first);
if (bitmap == null) {