Optimize AutoFillManager calls

AutoFillManager needs to handle calls for value changes and
reset only if there was a focus change reported, hence the
APIs for value change and reset do not need to connect to
the backing service to determine the enabled state.

Test: Auto-fill CTS tests pass

bug:35641424

Change-Id: I50e449925034c54257e35aa78e066e2a0fa53c03
This commit is contained in:
Svetoslav Ganov
2017-02-21 15:28:20 -08:00
parent cac64f616f
commit acfa5ee00a

View File

@@ -179,8 +179,6 @@ public final class AutoFillManager {
* @param view view whose focus changed.
*/
public void valueChanged(View view) {
ensureServiceClientAddedIfNeeded();
if (!mEnabled || !mHasSession) {
return;
}
@@ -199,8 +197,6 @@ public final class AutoFillManager {
* @param value new value of the child.
*/
public void virtualValueChanged(View parent, int childId, AutoFillValue value) {
ensureServiceClientAddedIfNeeded();
if (!mEnabled || !mHasSession) {
return;
}
@@ -216,8 +212,6 @@ public final class AutoFillManager {
* call this method after the form is submitted and another page is rendered.
*/
public void reset() {
ensureServiceClientAddedIfNeeded();
if (!mEnabled && !mHasSession) {
return;
}