PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fixed AutofillPreferenceCategory so it's UI is properly refreshed."

Browse Source
This commit is contained in:
TreeHugger Robot
2018-07-30 18:27:40 +00:00
committed by Android (Google) Code Review
parent 791b34b303 a305b779e8
commit 25c9375b6a
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/com/android/settings/development/autofill/AutofillPreferenceCategory.java
View File

@@ -19,6 +19,7 @@ import android.content.Context;
import android.database.ContentObserver;
import android.net.Uri;
import android.os.Handler;
import android.os.Looper;
import android.os.UserHandle;
import android.provider.Settings;
import android.util.AttributeSet;
@@ -30,22 +31,23 @@ import androidx.preference.PreferenceCategory;
public final class AutofillPreferenceCategory extends PreferenceCategory {
private static final String TAG = "AutofillPreferenceCategory";
private static final long DELAYED_MESSAGE_TIME_MS = 2000;
private final ContentResolver mContentResolver;
private final ContentObserver mSettingsObserver;
private final Handler mHandler = new Handler(Looper.getMainLooper());
public AutofillPreferenceCategory(Context context, AttributeSet attrs) {
super(context, attrs);
mSettingsObserver = new ContentObserver(new Handler()) {
mSettingsObserver = new ContentObserver(mHandler) {
@Override
public void onChange(boolean selfChange, Uri uri, int userId) {
Log.w(TAG, "Autofill Service changed, but UI cannot be refreshed");
// TODO(b/111838239): we cannot update the UI because AFM.isEnabled() will return
// the previous value. Once that's fixed, we'll need to call one of the 2 callbacks
// below:
// notifyChanged();
// notifyDependencyChange(shouldDisableDependents());
// We cannot apply the change yet because AutofillManager.isEnabled() state is
// updated by a ContentObserver as well and there's no guarantee of which observer
// is called first - hence, it's possible that the state didn't change here yet.
mHandler.postDelayed(() -> notifyDependencyChange(shouldDisableDependents()),
DELAYED_MESSAGE_TIME_MS);
}
};
mContentResolver = context.getContentResolver();