Merge "Avoid using JUnit code in production WallpaperCropper"

This commit is contained in:
Paul Duffin
2017-02-20 12:40:17 +00:00
committed by Gerrit Code Review
3 changed files with 12 additions and 10 deletions

View File

@@ -18,8 +18,6 @@ package com.android.gallery3d.glrenderer;
import android.graphics.Bitmap;
import junit.framework.Assert;
// BitmapTexture is a texture whose content is specified by a fixed Bitmap.
//
// The texture does not own the Bitmap. The user should make sure the Bitmap
@@ -34,7 +32,9 @@ public class BitmapTexture extends UploadedTexture {
public BitmapTexture(Bitmap bitmap, boolean hasBorder) {
super(hasBorder);
Assert.assertTrue(bitmap != null && !bitmap.isRecycled());
if (bitmap == null || bitmap.isRecycled()) {
throw new AssertionError();
}
mContentBitmap = bitmap;
}

View File

@@ -16,8 +16,6 @@
package com.android.gallery3d.glrenderer;
import junit.framework.Assert;
public class GLPaint {
private float mLineWidth = 1f;
private int mColor = 0;
@@ -31,7 +29,9 @@ public class GLPaint {
}
public void setLineWidth(float width) {
Assert.assertTrue(width >= 0);
if (width < 0) {
throw new AssertionError();
}
mLineWidth = width;
}

View File

@@ -20,8 +20,6 @@ import android.graphics.Bitmap;
import android.graphics.Bitmap.Config;
import android.opengl.GLUtils;
import junit.framework.Assert;
import java.util.HashMap;
import javax.microedition.khronos.opengles.GL11;
@@ -144,7 +142,9 @@ public abstract class UploadedTexture extends BasicTexture {
}
private void freeBitmap() {
Assert.assertTrue(mBitmap != null);
if (mBitmap == null) {
throw new AssertionError();
}
onFreeBitmap(mBitmap);
mBitmap = null;
}
@@ -219,7 +219,9 @@ public abstract class UploadedTexture extends BasicTexture {
int texWidth = getTextureWidth();
int texHeight = getTextureHeight();
Assert.assertTrue(bWidth <= texWidth && bHeight <= texHeight);
if (bWidth > texWidth || bHeight > texHeight) {
throw new AssertionError();
}
// Upload the bitmap to a new texture.
mId = canvas.getGLId().generateTexture();