Window.Callback перезаписывается панелью инструментов - PullRequest
7 голосов
/ 21 марта 2019

Я обнаружил, что в setActionBar метод из Activity:

public void setActionBar(@Nullable Toolbar toolbar) {
    final ActionBar ab = getActionBar();
    if (ab instanceof WindowDecorActionBar) {
        throw new IllegalStateException("This Activity already has an action bar supplied " +
                "by the window decor. Do not request Window.FEATURE_ACTION_BAR and set " +
                "android:windowActionBar to false in your theme to use a Toolbar instead.");
    }

    // If we reach here then we're setting a new action bar
    // First clear out the MenuInflater to make sure that it is valid for the new Action Bar
    mMenuInflater = null;

    // If we have an action bar currently, destroy it
    if (ab != null) {
        ab.onDestroy();
    }

    if (toolbar != null) {
        final ToolbarActionBar tbab = new ToolbarActionBar(toolbar, getTitle(), this);
        mActionBar = tbab;
        mWindow.setCallback(tbab.getWrappedWindowCallback());
    } else {
        mActionBar = null;
        // Re-set the original window callback since we may have already set a Toolbar wrapper
        mWindow.setCallback(this);
    }

    invalidateOptionsMenu();
}

Как видите, mWindow.setCallback(tbab.getWrappedWindowCallback()) вызывается с обратным вызовом. И если мы проверим реализацию ToolbarAractionBar:

ToolbarActionBar(Toolbar toolbar, CharSequence title, Window.Callback windowCallback) {
    mDecorToolbar = new ToolbarWidgetWrapper(toolbar, false);
    mWindowCallback = new ToolbarCallbackWrapper(windowCallback);
    mDecorToolbar.setWindowCallback(mWindowCallback);
    toolbar.setOnMenuItemClickListener(mMenuClicker);
    mDecorToolbar.setWindowTitle(title);
}

public Window.Callback getWrappedWindowCallback() {
    return mWindowCallback;
}

Оборачивает последний параметр конструктора, который был this (экземпляр Activity).

Проблема возникает, если перед вызовом setActionBar мы устанавливаем нашу собственную реализацию Window.Callback (которая правильно переносит оригинальную). В этом случае не наша реализация обернута ToolbarActionBar, а оригинальной.

Это не будет проблемой, если getWindow().getCallback() использовать вместо this в качестве ToolbarActionBar параметра конструктора. Так это выглядит как ошибка в Android SDK.

Вопрос : кто-нибудь знает, как справиться с такой ситуацией, или какой-то обходной путь для этой проблемы? (давайте предположим, что мы должны установить наш обратный вызов перед добавлением панели действий)


кстати. setSupportActionBar выглядит не одинаково, но очень похоже и имеет ту же проблему.

...