Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6c95f97a authored by Kurt Partridge's avatar Kurt Partridge
Browse files

ResearchLogging fix password field collection 

password data collection was too "sticky"; the next field wouldn't be collected either.

Change-Id: I739a956f663cff33216f83ac453b2a89d8c33dde
parent 359c35e0

java/src/com/android/inputmethod/research/ResearchLogger.java

+5 −2
Original line number Diff line number Diff line
@@ -60,6 +60,7 @@ import com.android.inputmethod.keyboard.MainKeyboardView; 
import com.android.inputmethod.latin.CollectionUtils;

import com.android.inputmethod.latin.Constants;

import com.android.inputmethod.latin.Dictionary;

import com.android.inputmethod.latin.InputTypeUtils;

import com.android.inputmethod.latin.LatinIME;

import com.android.inputmethod.latin.R;

import com.android.inputmethod.latin.RichInputConnection;
@@ -791,8 +792,11 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang 
    public static void latinIME_onStartInputViewInternal(final EditorInfo editorInfo,

            final SharedPreferences prefs) {

        final ResearchLogger researchLogger = getInstance();

        researchLogger.start();

        if (editorInfo != null) {

            final boolean isPassword = InputTypeUtils.isPasswordInputType(editorInfo.inputType)

                    || InputTypeUtils.isVisiblePasswordInputType(editorInfo.inputType);

            getInstance().setIsPasswordView(isPassword);

            researchLogger.start();

            final Context context = researchLogger.mInputMethodService;

            try {

                final PackageInfo packageInfo;
@@ -1059,7 +1063,6 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang 
                keyboard.mOccupiedHeight,

                keyboard.mKeys

            };

            getInstance().setIsPasswordView(isPasswordView);

            getInstance().enqueueEvent(EVENTKEYS_MAINKEYBOARDVIEW_SETKEYBOARD, values);

        }

    }