From c36ff4cf4f44b0e40a8a6c86f2c52652151ab9a4 Mon Sep 17 00:00:00 2001
From: Paul Duffin <paulduffin@google.com>
Date: Mon, 20 Feb 2017 12:33:42 +0000
Subject: [PATCH] Avoid using JUnit code in production WallpaperCropper

Bug: 30188076
Test: make checkbuild
Change-Id: Ib6fe445a246d15dbabb8b86f9251b0059ed216b5
---
 .../android/gallery3d/glrenderer/BitmapTexture.java    |  6 +++---
 .../src/com/android/gallery3d/glrenderer/GLPaint.java  |  6 +++---
 .../android/gallery3d/glrenderer/UploadedTexture.java  | 10 ++++++----
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/BitmapTexture.java b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/BitmapTexture.java
index 100b0b3b9c1f8..e708d53e556cf 100644
--- a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/BitmapTexture.java
+++ b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/BitmapTexture.java
@@ -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;
     }
 
diff --git a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/GLPaint.java b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/GLPaint.java
index 16b220690ef40..358b2dc366939 100644
--- a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/GLPaint.java
+++ b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/GLPaint.java
@@ -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;
     }
 
diff --git a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/UploadedTexture.java b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/UploadedTexture.java
index f41a979b7ddd0..2e878791e7fee 100644
--- a/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/UploadedTexture.java
+++ b/packages/WallpaperCropper/src/com/android/gallery3d/glrenderer/UploadedTexture.java
@@ -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();