Refactoring useless UserUtils class

Bug: 16542609
Change-Id: Iadb83c5e378cd342ea89d6e224ddc45e6f940c8b
This commit is contained in:
Alexandra Gherghina
2014-07-24 19:13:25 +01:00
parent 72f7072328
commit 95b86a5488
7 changed files with 19 additions and 44 deletions

View File

@@ -68,6 +68,7 @@ import android.widget.TabWidget;
import com.android.settings.dashboard.DashboardCategory; import com.android.settings.dashboard.DashboardCategory;
import com.android.settings.dashboard.DashboardTile; import com.android.settings.dashboard.DashboardTile;
import com.android.settings.users.CircleFramedDrawable;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
@@ -779,4 +780,14 @@ public final class Utils {
context.getSystemService(Context.PERSISTENT_DATA_BLOCK_SERVICE); context.getSystemService(Context.PERSISTENT_DATA_BLOCK_SERVICE);
manager.setOemUnlockEnabled(enabled); manager.setOemUnlockEnabled(enabled);
} }
/**
* Returns a circular icon for a user.
*/
public static Drawable getUserIcon(Context context, UserManager um, UserInfo user) {
if (user.iconPath == null) return null;
Bitmap icon = um.getUserIcon(user.id);
if (icon == null) return null;
return CircleFramedDrawable.getInstance(context, icon);
}
} }

View File

@@ -17,7 +17,7 @@
package com.android.settings.applications; package com.android.settings.applications;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.users.UserUtils; import com.android.settings.Utils;
import android.app.ActivityManager; import android.app.ActivityManager;
import android.app.ActivityManagerNative; import android.app.ActivityManagerNative;
@@ -837,8 +837,7 @@ public class RunningState {
UserInfo info = mUm.getUserInfo(newItem.mUserId); UserInfo info = mUm.getUserInfo(newItem.mUserId);
userItem.mUser.mInfo = info; userItem.mUser.mInfo = info;
if (info != null) { if (info != null) {
userItem.mUser.mIcon = UserUtils.getUserIcon(context, mUm, userItem.mUser.mIcon = Utils.getUserIcon(context, mUm, info);
info, context.getResources());
} }
String name = info != null ? info.name : null; String name = info != null ? info.name : null;
if (name == null && info != null) { if (name == null && info != null) {

View File

@@ -25,9 +25,10 @@ import android.graphics.drawable.Drawable;
import android.os.BatteryStats; import android.os.BatteryStats;
import android.os.Handler; import android.os.Handler;
import android.os.UserManager; import android.os.UserManager;
import com.android.internal.os.BatterySipper; import com.android.internal.os.BatterySipper;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.users.UserUtils; import com.android.settings.Utils;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.HashMap; import java.util.HashMap;
@@ -160,7 +161,7 @@ public class BatteryEntry {
case USER: { case USER: {
UserInfo info = um.getUserInfo(sipper.userId); UserInfo info = um.getUserInfo(sipper.userId);
if (info != null) { if (info != null) {
icon = UserUtils.getUserIcon(context, um, info, context.getResources()); icon = Utils.getUserIcon(context, um, info);
name = info != null ? info.name : null; name = info != null ? info.name : null;
if (name == null) { if (name == null) {
name = Integer.toString(info.id); name = Integer.toString(info.id);

View File

@@ -32,7 +32,6 @@ import android.util.SparseArray;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.Utils; import com.android.settings.Utils;
import com.android.settings.users.UserUtils;
/** /**
* Return details about a specific UID, handling special cases like * Return details about a specific UID, handling special cases like
@@ -121,7 +120,7 @@ public class UidDetailProvider {
final UserInfo info = um.getUserInfo(userHandle); final UserInfo info = um.getUserInfo(userHandle);
if (info != null) { if (info != null) {
detail.label = res.getString(R.string.running_process_item_user_label, info.name); detail.label = res.getString(R.string.running_process_item_user_label, info.name);
detail.icon = UserUtils.getUserIcon(mContext, um, info, res); detail.icon = Utils.getUserIcon(mContext, um, info);
return detail; return detail;
} }
} }

View File

@@ -37,7 +37,7 @@ import com.android.settings.R;
* Converts the user avatar icon to a circularly clipped one. * Converts the user avatar icon to a circularly clipped one.
* TODO: Move this to an internal framework class and share with the one in Keyguard. * TODO: Move this to an internal framework class and share with the one in Keyguard.
*/ */
class CircleFramedDrawable extends Drawable { public class CircleFramedDrawable extends Drawable {
private final Bitmap mBitmap; private final Bitmap mBitmap;
private final int mSize; private final int mSize;

View File

@@ -126,8 +126,7 @@ public class EditUserInfoController {
final ImageView userPhotoView = (ImageView) content.findViewById(R.id.user_photo); final ImageView userPhotoView = (ImageView) content.findViewById(R.id.user_photo);
Drawable drawable = null; Drawable drawable = null;
if (mSavedPhoto != null) { if (mSavedPhoto != null) {
drawable = CircleFramedDrawable.getInstance(activity, drawable = CircleFramedDrawable.getInstance(activity, mSavedPhoto);
mSavedPhoto);
} else { } else {
drawable = currentUserIcon; drawable = currentUserIcon;
if (drawable == null) { if (drawable == null) {

View File

@@ -1,34 +0,0 @@
/*
* Copyright (C) 2012 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.settings.users;
import android.content.Context;
import android.content.pm.UserInfo;
import android.content.res.Resources;
import android.graphics.Bitmap;
import android.graphics.drawable.BitmapDrawable;
import android.graphics.drawable.Drawable;
import android.os.UserManager;
public class UserUtils {
public static Drawable getUserIcon(Context context, UserManager um, UserInfo user, Resources res) {
if (user.iconPath == null) return null;
Bitmap icon = um.getUserIcon(user.id);
if (icon == null) return null;
return CircleFramedDrawable.getInstance(context, icon);
}
}