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

Commit 66837204 authored by Matt Sarett's avatar Matt Sarett Committed by Android (Google) Code Review
Browse files

Merge "Clean-ups for BitmapRegionDecoder"

parents 6e690ac8 d31512b9

core/jni/android/graphics/BitmapFactory.cpp

+1 −1
Original line number Diff line number Diff line
@@ -265,7 +265,7 @@ static jobject doDecode(JNIEnv* env, SkStreamRewindable* stream, jobject padding 
    if (options != NULL) {

        jstring mimeType = encodedFormatToString(env, codec->getEncodedFormat());

        if (env->ExceptionCheck()) {

            return nullObjectReturn("OOM in getEncodedFormat()");

            return nullObjectReturn("OOM in encodedFormatToString()");

        }

        env->SetIntField(options, gOptions_widthFieldID, size.width());

        env->SetIntField(options, gOptions_heightFieldID, size.height());

core/jni/android/graphics/BitmapRegionDecoder.cpp

+3 −3
Original line number Diff line number Diff line
@@ -136,9 +136,6 @@ static jobject nativeDecodeRegion(JNIEnv* env, jobject, jlong brdHandle, jint in 
        sampleSize = env->GetIntField(options, gOptions_sampleSizeFieldID);

        jobject jconfig = env->GetObjectField(options, gOptions_configFieldID);

        colorType = GraphicsJNI::getNativeBitmapColorType(env, jconfig);

        if (kAlpha_8_SkColorType == colorType) {

            colorType = kGray_8_SkColorType;

        }

        requireUnpremul = !env->GetBooleanField(options, gOptions_premultipliedFieldID);

        javaBitmap = env->GetObjectField(options, gOptions_bitmapFieldID);

        // The Java options of ditherMode and preferQualityOverSpeed are deprecated.  We will
@@ -189,6 +186,9 @@ static jobject nativeDecodeRegion(JNIEnv* env, jobject, jlong brdHandle, jint in 
        env->SetIntField(options, gOptions_heightFieldID, bitmap.height());

        env->SetObjectField(options, gOptions_mimeFieldID,

                encodedFormatToString(env, brd->getEncodedFormat()));

        if (env->ExceptionCheck()) {

            return nullObjectReturn("OOM in encodedFormatToString()");

        }

    }



    // If we may have reused a bitmap, we need to indicate that the pixels have changed.