Merge pull request #22645 from brave/autofill_ignore_omnibox_android
[Android] ignore omnibox in autofill service
This commit is contained in:
+15
-1
@@ -20,6 +20,7 @@ import android.support.annotation.Nullable;
|
||||
import android.view.View;
|
||||
import android.view.autofill.AutofillId;
|
||||
import android.view.autofill.AutofillValue;
|
||||
import android.view.inputmethod.EditorInfo;
|
||||
import android.widget.RemoteViews;
|
||||
|
||||
import androidx.annotation.NonNull;
|
||||
@@ -148,7 +149,6 @@ public class BraveAutofillBackgroundServiceImpl extends ChromeBackgroundServiceI
|
||||
PersonalDataManager.getInstance().getProfilesToSuggest(true);
|
||||
FillResponse.Builder fillResponse = new FillResponse.Builder();
|
||||
if (profileList != null && !profileList.isEmpty()) {
|
||||
|
||||
for (AutofillProfile profile : profileList) {
|
||||
|
||||
Map<AutofillId, String> autofillDataMap = new HashMap<>();
|
||||
@@ -387,6 +387,10 @@ public class BraveAutofillBackgroundServiceImpl extends ChromeBackgroundServiceI
|
||||
return null;
|
||||
}
|
||||
|
||||
if (isOmnibox(node, hint)) {
|
||||
return null;
|
||||
}
|
||||
|
||||
if (hint.contains(context.getResources().getString(R.string.password))) {
|
||||
return View.AUTOFILL_HINT_PASSWORD;
|
||||
}
|
||||
@@ -443,6 +447,16 @@ public class BraveAutofillBackgroundServiceImpl extends ChromeBackgroundServiceI
|
||||
return null;
|
||||
}
|
||||
|
||||
private boolean isOmnibox(ViewNode node, String hint) {
|
||||
|
||||
int inputType = node.getInputType();
|
||||
if ((inputType & EditorInfo.TYPE_TEXT_VARIATION_URI)
|
||||
== EditorInfo.TYPE_TEXT_VARIATION_URI) {
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
private boolean isAutofillAddress(Map<String, AutofillId> fields) {
|
||||
return fields.containsKey(View.AUTOFILL_HINT_POSTAL_ADDRESS)
|
||||
|| fields.containsKey(View.AUTOFILL_HINT_POSTAL_CODE);
|
||||
|
||||
Reference in New Issue
Block a user