Merge "Fix discovered logging issues for Autofill." into udc-dev
This commit is contained in:
@@ -74,9 +74,6 @@ public final class FillRequestEventLogger {
|
|||||||
public static final int TRIGGER_REASON_SERVED_FROM_CACHED_RESPONSE =
|
public static final int TRIGGER_REASON_SERVED_FROM_CACHED_RESPONSE =
|
||||||
AUTOFILL_FILL_REQUEST_REPORTED__REQUEST_TRIGGER_REASON__TRIGGER_REASON_SERVED_FROM_CACHED_RESPONSE;
|
AUTOFILL_FILL_REQUEST_REPORTED__REQUEST_TRIGGER_REASON__TRIGGER_REASON_SERVED_FROM_CACHED_RESPONSE;
|
||||||
|
|
||||||
// Augmented autofill currently doesn't have an assigned request_id, use -2 as the magic number.
|
|
||||||
public static final int AUGMENTED_AUTOFILL_REQUEST_ID = -2;
|
|
||||||
|
|
||||||
private final int mSessionId;
|
private final int mSessionId;
|
||||||
private Optional<FillRequestEventInternal> mEventInternal;
|
private Optional<FillRequestEventInternal> mEventInternal;
|
||||||
|
|
||||||
|
|||||||
@@ -1158,7 +1158,7 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
|
|||||||
}
|
}
|
||||||
mSessionFlags.mAugmentedAutofillOnly = true;
|
mSessionFlags.mAugmentedAutofillOnly = true;
|
||||||
mFillRequestEventLogger.maybeSetRequestId(AUGMENTED_AUTOFILL_REQUEST_ID);
|
mFillRequestEventLogger.maybeSetRequestId(AUGMENTED_AUTOFILL_REQUEST_ID);
|
||||||
mFillRequestEventLogger.maybeSetIsAugmented(mSessionFlags.mAugmentedAutofillOnly);
|
mFillRequestEventLogger.maybeSetIsAugmented(true);
|
||||||
mFillRequestEventLogger.logAndEndEvent();
|
mFillRequestEventLogger.logAndEndEvent();
|
||||||
triggerAugmentedAutofillLocked(flags);
|
triggerAugmentedAutofillLocked(flags);
|
||||||
return;
|
return;
|
||||||
@@ -4818,6 +4818,7 @@ final class Session implements RemoteFillService.FillServiceCallbacks, ViewState
|
|||||||
mFillRequestEventLogger.maybeSetAppPackageUid(uid);
|
mFillRequestEventLogger.maybeSetAppPackageUid(uid);
|
||||||
mFillRequestEventLogger.maybeSetFlags(mFlags);
|
mFillRequestEventLogger.maybeSetFlags(mFlags);
|
||||||
mFillRequestEventLogger.maybeSetRequestId(AUGMENTED_AUTOFILL_REQUEST_ID);
|
mFillRequestEventLogger.maybeSetRequestId(AUGMENTED_AUTOFILL_REQUEST_ID);
|
||||||
|
mFillRequestEventLogger.maybeSetIsAugmented(true);
|
||||||
mFillRequestEventLogger.logAndEndEvent();
|
mFillRequestEventLogger.logAndEndEvent();
|
||||||
|
|
||||||
final ViewState viewState = mViewStates.get(mCurrentViewId);
|
final ViewState viewState = mViewStates.get(mCurrentViewId);
|
||||||
|
|||||||
Reference in New Issue
Block a user