Ошибка с методом Android 4.3 ImageView getImageMatrix()

Я нашел то, что я считаю проблемой. Я взглянул на исходный код для ImageView и обнаружил, что setImageMatrix(Matrix matrix) сохраняет матрицу в другом поле, чем возвращает getImageMatrix()...

Android 4.4 ImageView

public void setImageMatrix(Matrix matrix) {
    // collaps null and identity to just null
    if (matrix != null && matrix.isIdentity()) {
        matrix = null;
    }

    // don't invalidate unless we're actually changing our matrix
    if (matrix == null && !mMatrix.isIdentity() ||
            matrix != null && !mMatrix.equals(matrix)) {
        mMatrix.set(matrix);
        configureBounds();
        invalidate();
    }
}

Здесь матрица хранится в поле mMatrix

public Matrix getImageMatrix() {
    if (mDrawMatrix == null) { //<-- should be mMatrix == null
        return new Matrix(Matrix.IDENTITY_MATRIX);
    }
    return mDrawMatrix; //<-- NOT THE RIGHT FIELD TO RETURN
}

Пока getImageMatrix() возвращает mDrawMatrix...

Android 4.1.2 ImageView

public Matrix getImageMatrix() {
    return mMatrix;
}

public void setImageMatrix(Matrix matrix) {
    // collaps null and identity to just null
    if (matrix != null && matrix.isIdentity()) {
        matrix = null;
    }

    // don't invalidate unless we're actually changing our matrix
    if (matrix == null && !mMatrix.isIdentity() ||
            matrix != null && !mMatrix.equals(matrix)) {
        mMatrix.set(matrix);
        configureBounds();
        invalidate();
    }
}

оба метода используют одно и то же поле – mMatrix

Итак, проблема прямо здесь --- вдруг getImageMatrix() возвращает неправильное поле...

Хотя мой предыдущий ответ описывает общую проблему, на самом деле это может быть не ошибка. Как указано в Generic Holiday Name, mDrawMatrix должно быть установлено на mMatrix в методе configureBounds(), что устранит эту проблему/ошибку/что угодно.

ОДНАКО мне пришлось добавить несколько строк кода, чтобы configureBounds() действительно заработало:

private void configureBounds() {
    if (mDrawable == null || !mHaveFrame) {
        return;
    }

    int dwidth = mDrawableWidth;
    int dheight = mDrawableHeight;

    ...

    if (dwidth <= 0 || dheight <= 0 || ScaleType.FIT_XY == mScaleType) {
        /* If the drawable has no intrinsic size, or we're told to
            scaletofit, then we just fill our entire view.
        */
        mDrawable.setBounds(0, 0, vwidth, vheight);
        mDrawMatrix = null;
    } else {
             //here's the where mDrawMatrix == mMatrix IF using scaleType.MATRIX
             ...
    }
}

Итак, чтобы заставить это работать так, как я ожидаю, с 4.2 и более ранними версиями, вам необходимо убедиться:

1. mDrawable != null. Раньше это не было проблемой, но в моем случае я не использовал рисование, поэтому все терпело неудачу (оператор return был выполнен сразу)
2. 'dwidth >0 && dheight >0. Это не проблема, если у вас есть настоящий рисунок, но, как я уже сказал, у меня его не было.
3. mHaveFrame = true. Я понятия не имел, что это такое - никогда не использовал его. Единственный способ установить это значение true — вызвать setFrame(int, int, int, int).

Чтобы мой код масштабирования снова заработал, мне пришлось добавить следующее:

 //potentially a fix for the "bug" that appears in Android 4.3+
 //mDrawable cannot be null anymore for getImageMatrix to work---stupid
 //therefore the imageview class MUST set a drawable for matrix scaling to work
 //so here I am using a "empty" drawable to get around this
 ShapeDrawable fakeDrawable = new ShapeDrawable(); //so mDrawable != null
 fakeDrawable.setIntrinsicHeight(1); //so dwidth and dheight are > 0
 fakeDrawable.setIntrinsicWidth(1);

 setImageDrawable(fakeDrawable);

 setFrame(0, 0, viewWidth, viewHeight); //setting a frame so mHaveFrame = true

УРА