Chromium: Make password saving option independent from autofill option in Privacy settings

[haanhvu]

Chromium: Make password saving option independent from autofill option in Privacy settings

Currently in Privacy settings in Chromium backend, the password saving option depends on the autofill option. If autofill is disabled, password saving doesn't work whether enabled or not. This is because Chromium doesn't provide an autofill option but provides a login selection prompt that asks users whether to autofill the saved password or not. This commit makes password saving option independent from autofill option. The autofill option now only affects whether to show login selection prompt.

Fixes #1707

[svillar]

I have to admit that I don't really understand what the problem is. I don't see the relationship between password saving and autofill. Actually if you check the implementation of PromptDelegate.java the onLoginSave method does depend on LoginAutocompleteEnabled not in AutoFillEnabled. So I'm a bit lost about why this is needed and whether this fixes anything

[haanhvu]

I have to admit that I don't really understand what the problem is. I don't see the relationship between password saving and autofill. Actually if you check the implementation of PromptDelegate.java the onLoginSave method does depend on LoginAutocompleteEnabled not in AutoFillEnabled. So I'm a bit lost about why this is needed and whether this fixes anything

As stated in #1707, the problem is if users enable password saving and disable autofill, password saving dialog doesn't show. And this only happens in Chromium. You could reproduce this with the main branch.

You're right about onLoginSave depending on LoginAutocompleteEnabled not AutoFillEnabled. However in Chromium implementation, onLoginSave is called by saveOrUpdatePassword in PromptDelegateImpl.java:

wolvic/app/src/common/chromium/com/igalia/wolvic/browser/api/impl/PromptDelegateImpl.java

Line 997 in 9b1e83d

mDelegate.onLoginSave(mSession, mLoginSavePrompt);

This is in turn called by WolvicPasswordManagerClient in native code: https://github.com/Igalia/wolvic-chromium/blob/40804fadc808686bdebc54a71196c4b9a3ff68c7/wolvic/browser/autocomplete/wolvic_password_manager_client.cc#L164

However, password manager client checks if saving and autofill enabled too, as I stated in #1707 (comment)

Does this make sense to you? I don't know if I missed something, but I checked my change on Quest 3 and it fixed the issue. When I enabled password saving and disabled autofill, password saving dialog now showed.

Also, unlike Gecko, Chromium shows an autofill prompt to ask if you want to autofill in every login after saving. I made this dependent on the autofill option in the Settings now. If users disable autofill, this autofill prompt now doesn't show.