Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cc6855fc authored by Chris Craik's avatar Chris Craik Committed by Android (Google) Code Review
Browse files

Merge "Revert "Avoid throwing ISE in Canvas#restore underflow""

parents 7298d1f0 7d5dda4d

core/jni/android_graphics_Canvas.cpp

+2 −2
Original line number Diff line number Diff line
@@ -89,7 +89,7 @@ static jint saveLayerAlpha(JNIEnv* env, jobject, jlong canvasHandle, jfloat l, j 
static void restore(JNIEnv* env, jobject, jlong canvasHandle) {

    Canvas* canvas = get_canvas(canvasHandle);

    if (canvas->getSaveCount() <= 1) {  // cannot restore anymore

        // fail silently on underflow, so as not to break existing apps that miscount

        doThrowISE(env, "Underflow in restore - more restores than saves");

        return;

    }

    canvas->restore();
@@ -98,7 +98,7 @@ static void restore(JNIEnv* env, jobject, jlong canvasHandle) { 
static void restoreToCount(JNIEnv* env, jobject, jlong canvasHandle, jint restoreCount) {

    Canvas* canvas = get_canvas(canvasHandle);

    if (restoreCount < 1 || restoreCount > canvas->getSaveCount()) {

        // fail silently on underflow, so as not to break existing apps that miscount

        doThrowIAE(env, "Underflow in restoreToCount - more restores than saves");

        return;

    }

    canvas->restoreToCount(restoreCount);