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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/snackbar/smartlockautosignin/AutoSigninSnackbarController.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/snackbar/smartlockautosignin/AutoSigninSnackbarController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/snackbar/smartlockautosignin/AutoSigninSnackbarController.java b/chrome/android/java/src/org/chromium/chrome/browser/snackbar/smartlockautosignin/AutoSigninSnackbarController.java
index bc6f562d8e47b34cd00c383fb18a7079eb9f6f5c..6d6bf235b347aba16f2205e50f183661da0b5d57 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/snackbar/smartlockautosignin/AutoSigninSnackbarController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/snackbar/smartlockautosignin/AutoSigninSnackbarController.java
@@ -4,9 +4,11 @@
package org.chromium.chrome.browser.snackbar.smartlockautosignin;
-import android.content.res.Resources;
+import android.content.Context;
import android.graphics.Bitmap;
-import android.graphics.BitmapFactory;
+import android.graphics.Canvas;
+import android.graphics.drawable.Drawable;
+import android.support.v7.content.res.AppCompatResources;
import org.chromium.base.ApiCompatibilityUtils;
import org.chromium.base.annotations.CalledByNative;
@@ -40,10 +42,19 @@ public class AutoSigninSnackbarController
new AutoSigninSnackbarController(snackbarManager, tab);
Snackbar snackbar = Snackbar.make(text, snackbarController, Snackbar.TYPE_NOTIFICATION,
Snackbar.UMA_AUTO_LOGIN);
- Resources resources = tab.getWindowAndroid().getActivity().get().getResources();
- int backgroundColor = ApiCompatibilityUtils.getColor(resources, R.color.light_active_color);
- Bitmap icon = BitmapFactory.decodeResource(
- resources, R.drawable.account_management_no_picture);
+ Context context = tab.getWindowAndroid().getActivity().get();
+ int backgroundColor =
+ ApiCompatibilityUtils.getColor(context.getResources(), R.color.light_active_color);
+
+ Drawable drawableIcon =
+ AppCompatResources.getDrawable(context, R.drawable.logo_avatar_anonymous);
+ int imageSizePx =
+ context.getResources().getDimensionPixelOffset(R.dimen.snackbar_image_size);
+ Bitmap icon = Bitmap.createBitmap(imageSizePx, imageSizePx, Bitmap.Config.ARGB_8888);
+ Canvas canvas = new Canvas(icon);
+ drawableIcon.setBounds(0, 0, canvas.getWidth(), canvas.getHeight());
+ drawableIcon.draw(canvas);
+
snackbar.setSingleLine(false).setBackgroundColor(backgroundColor).setProfileImage(icon);
Theresa 2017/06/01 23:22:15 It looks like this is the only class that uses the
bsazonov 2017/06/02 13:17:20 Done.
snackbarManager.showSnackbar(snackbar);
}

Powered by Google App Engine
This is Rietveld 408576698