Merge "Revert "WallpaperColors hint computation"" into pi-dev
am: cf2c35dca6
Change-Id: Ibc3760ce650c8d48f261e6486edb51582391b710
This commit is contained in:
@@ -144,13 +144,6 @@ public final class WallpaperColors implements Parcelable {
|
||||
* @param bitmap Source where to extract from.
|
||||
*/
|
||||
public static WallpaperColors fromBitmap(@NonNull Bitmap bitmap) {
|
||||
return fromBitmap(bitmap, false /* computeHints */);
|
||||
}
|
||||
|
||||
/**
|
||||
* @hide
|
||||
*/
|
||||
public static WallpaperColors fromBitmap(@NonNull Bitmap bitmap, boolean computeHints) {
|
||||
if (bitmap == null) {
|
||||
throw new IllegalArgumentException("Bitmap can't be null");
|
||||
}
|
||||
@@ -200,7 +193,7 @@ public final class WallpaperColors implements Parcelable {
|
||||
}
|
||||
}
|
||||
|
||||
int hints = computeHints ? calculateDarkHints(bitmap) : 0;
|
||||
int hints = calculateDarkHints(bitmap);
|
||||
|
||||
if (shouldRecycle) {
|
||||
bitmap.recycle();
|
||||
|
||||
@@ -557,7 +557,7 @@ public class WallpaperManagerService extends IWallpaperManager.Stub
|
||||
if (cropFile != null) {
|
||||
Bitmap bitmap = BitmapFactory.decodeFile(cropFile);
|
||||
if (bitmap != null) {
|
||||
colors = WallpaperColors.fromBitmap(bitmap, true /* computeHints */);
|
||||
colors = WallpaperColors.fromBitmap(bitmap);
|
||||
bitmap.recycle();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -20,8 +20,6 @@ import static org.junit.Assert.assertEquals;
|
||||
import static org.junit.Assert.assertTrue;
|
||||
|
||||
import android.app.WallpaperColors;
|
||||
import android.graphics.Color;
|
||||
import android.graphics.drawable.ColorDrawable;
|
||||
import android.os.Handler;
|
||||
import android.os.Message;
|
||||
import android.os.SystemClock;
|
||||
@@ -87,17 +85,4 @@ public class WallpaperServiceTests {
|
||||
assertEquals("OnComputeColors should have been deferred.",
|
||||
0, eventCountdown.getCount());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testFromDrawableTest_doesntComputeHints() {
|
||||
WallpaperColors wallpaperColors = WallpaperColors.fromDrawable(
|
||||
new ColorDrawable(Color.BLACK));
|
||||
assertEquals("WallpaperColors should not support dark theme.", 0,
|
||||
wallpaperColors.getColorHints() & WallpaperColors.HINT_SUPPORTS_DARK_THEME);
|
||||
|
||||
wallpaperColors = WallpaperColors.fromDrawable(
|
||||
new ColorDrawable(Color.WHITE));
|
||||
assertEquals("WallpaperColors should not support dark text.", 0,
|
||||
wallpaperColors.getColorHints() & WallpaperColors.HINT_SUPPORTS_DARK_TEXT);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user