am 0dac17f5: Merge "Use dedicated DictionaryFacilitator for DistracterFilter"

* commit '0dac17f5fb2cae8f258cd06c8ac7cb8d94ebdaa0':
  Use dedicated DictionaryFacilitator for DistracterFilter
main
Keisuke Kuroyanagi 2014-05-16 10:52:01 +00:00 committed by Android Git Automerger
commit 53990689dc
4 changed files with 55 additions and 15 deletions

View File

@ -140,10 +140,10 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
private final SubtypeState mSubtypeState = new SubtypeState(); private final SubtypeState mSubtypeState = new SubtypeState();
// Object for reacting to adding/removing a dictionary pack. // Object for reacting to adding/removing a dictionary pack.
private BroadcastReceiver mDictionaryPackInstallReceiver = private final BroadcastReceiver mDictionaryPackInstallReceiver =
new DictionaryPackInstallBroadcastReceiver(this); new DictionaryPackInstallBroadcastReceiver(this);
private BroadcastReceiver mDictionaryDumpBroadcastReceiver = private final BroadcastReceiver mDictionaryDumpBroadcastReceiver =
new DictionaryDumpBroadcastReceiver(this); new DictionaryDumpBroadcastReceiver(this);
private AlertDialog mOptionsDialog; private AlertDialog mOptionsDialog;
@ -1629,7 +1629,8 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
// boolean onKeyMultiple(final int keyCode, final int count, final KeyEvent event); // boolean onKeyMultiple(final int keyCode, final int count, final KeyEvent event);
// receive ringer mode change and network state change. // receive ringer mode change and network state change.
private BroadcastReceiver mConnectivityAndRingerModeChangeReceiver = new BroadcastReceiver() { private final BroadcastReceiver mConnectivityAndRingerModeChangeReceiver =
new BroadcastReceiver() {
@Override @Override
public void onReceive(final Context context, final Intent intent) { public void onReceive(final Context context, final Intent intent) {
final String action = intent.getAction(); final String action = intent.getAction();
@ -1746,8 +1747,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
@UsedForTesting @UsedForTesting
/* package for test */ DistracterFilter createDistracterFilter() { /* package for test */ DistracterFilter createDistracterFilter() {
return DistracterFilterUtils.createDistracterFilter( return DistracterFilterUtils.createDistracterFilter(this /* Context */, mKeyboardSwitcher);
mInputLogic.mSuggest, mKeyboardSwitcher);
} }
public void dumpDictionaryForDebug(final String dictName) { public void dumpDictionaryForDebug(final String dictName) {

View File

@ -16,8 +16,17 @@
package com.android.inputmethod.latin.utils; package com.android.inputmethod.latin.utils;
import java.util.Locale;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;
import android.content.Context;
import android.util.Log;
import com.android.inputmethod.keyboard.Keyboard; import com.android.inputmethod.keyboard.Keyboard;
import com.android.inputmethod.latin.Constants; import com.android.inputmethod.latin.Constants;
import com.android.inputmethod.latin.DictionaryFacilitatorForSuggest.
DictionaryInitializationListener;
import com.android.inputmethod.latin.Suggest; import com.android.inputmethod.latin.Suggest;
import com.android.inputmethod.latin.Suggest.OnGetSuggestedWordsCallback; import com.android.inputmethod.latin.Suggest.OnGetSuggestedWordsCallback;
import com.android.inputmethod.latin.SuggestedWords; import com.android.inputmethod.latin.SuggestedWords;
@ -29,6 +38,11 @@ import com.android.inputmethod.latin.WordComposer;
* or user history dictionaries * or user history dictionaries
*/ */
public class DistracterFilter { public class DistracterFilter {
private static final String TAG = DistracterFilter.class.getSimpleName();
private static final long TIMEOUT_TO_WAIT_LOADING_DICTIONARIES_IN_SECONDS = 120;
private final Context mContext;
private final Suggest mSuggest; private final Suggest mSuggest;
private final Keyboard mKeyboard; private final Keyboard mKeyboard;
@ -42,13 +56,13 @@ public class DistracterFilter {
/** /**
* Create a DistracterFilter instance. * Create a DistracterFilter instance.
* *
* @param suggest an instance of Suggest which will be used to obtain a list of suggestions * @param context the context.
* for a potential distracter
* @param keyboard the keyboard that is currently being used. This information is needed * @param keyboard the keyboard that is currently being used. This information is needed
* when calling mSuggest.getSuggestedWords(...) to obtain a list of suggestions. * when calling mSuggest.getSuggestedWords(...) to obtain a list of suggestions.
*/ */
public DistracterFilter(final Suggest suggest, final Keyboard keyboard) { public DistracterFilter(final Context context, final Keyboard keyboard) {
mSuggest = suggest; mContext = context;
mSuggest = new Suggest();
mKeyboard = keyboard; mKeyboard = keyboard;
} }
@ -66,19 +80,44 @@ public class DistracterFilter {
return false; return false;
} }
private void loadDictionariesForLocale(final Locale newlocale) throws InterruptedException {
final CountDownLatch countdownLatch = new CountDownLatch(1 /* count */);
mSuggest.mDictionaryFacilitator.resetDictionaries(mContext, newlocale,
false /* useContactsDict */, false /* usePersonalizedDicts */,
false /* forceReloadMainDictionary */,
new DictionaryInitializationListener() {
@Override
public void onUpdateMainDictionaryAvailability(
boolean isMainDictionaryAvailable) {
countdownLatch.countDown();
}
});
countdownLatch.await(TIMEOUT_TO_WAIT_LOADING_DICTIONARIES_IN_SECONDS, TimeUnit.SECONDS);
}
/** /**
* Determine whether a word is a distracter to words in dictionaries. * Determine whether a word is a distracter to words in dictionaries.
* *
* @param prevWord the previous word, or null if none. * @param prevWord the previous word, or null if none.
* @param testedWord the word that will be tested to see whether it is a distracter to words * @param testedWord the word that will be tested to see whether it is a distracter to words
* in dictionaries. * in dictionaries.
* @param locale the locale of words.
* @return true if testedWord is a distracter, otherwise false. * @return true if testedWord is a distracter, otherwise false.
*/ */
public boolean isDistracterToWordsInDictionaries(final String prevWord, public boolean isDistracterToWordsInDictionaries(final String prevWord,
final String testedWord) { final String testedWord, final Locale locale) {
if (mSuggest == null || mKeyboard == null) { if (mKeyboard == null || locale == null) {
return false; return false;
} }
if (!locale.equals(mSuggest.mDictionaryFacilitator.getLocale())) {
// Reset dictionaries for the locale.
try {
loadDictionariesForLocale(locale);
} catch (final InterruptedException e) {
Log.e(TAG, "Interrupted while waiting for loading dicts in DistracterFilter", e);
return false;
}
}
final WordComposer composer = new WordComposer(); final WordComposer composer = new WordComposer();
final int[] codePoints = StringUtils.toCodePointArray(testedWord); final int[] codePoints = StringUtils.toCodePointArray(testedWord);

View File

@ -16,17 +16,18 @@
package com.android.inputmethod.latin.utils; package com.android.inputmethod.latin.utils;
import android.content.Context;
import com.android.inputmethod.keyboard.Keyboard; import com.android.inputmethod.keyboard.Keyboard;
import com.android.inputmethod.keyboard.KeyboardSwitcher; import com.android.inputmethod.keyboard.KeyboardSwitcher;
import com.android.inputmethod.keyboard.MainKeyboardView; import com.android.inputmethod.keyboard.MainKeyboardView;
import com.android.inputmethod.latin.Suggest;
public class DistracterFilterUtils { public class DistracterFilterUtils {
private DistracterFilterUtils() { private DistracterFilterUtils() {
// This utility class is not publicly instantiable. // This utility class is not publicly instantiable.
} }
public static final DistracterFilter createDistracterFilter(final Suggest suggest, public static final DistracterFilter createDistracterFilter(final Context context,
final KeyboardSwitcher keyboardSwitcher) { final KeyboardSwitcher keyboardSwitcher) {
final MainKeyboardView mainKeyboardView = keyboardSwitcher.getMainKeyboardView(); final MainKeyboardView mainKeyboardView = keyboardSwitcher.getMainKeyboardView();
// TODO: Create Keyboard when mainKeyboardView is null. // TODO: Create Keyboard when mainKeyboardView is null.
@ -34,7 +35,7 @@ public class DistracterFilterUtils {
// spellchecker's logic. // spellchecker's logic.
final Keyboard keyboard = (mainKeyboardView != null) ? final Keyboard keyboard = (mainKeyboardView != null) ?
mainKeyboardView.getKeyboard() : null; mainKeyboardView.getKeyboard() : null;
final DistracterFilter distracterFilter = new DistracterFilter(suggest, keyboard); final DistracterFilter distracterFilter = new DistracterFilter(context, keyboard);
return distracterFilter; return distracterFilter;
} }
} }

View File

@ -150,7 +150,7 @@ public final class LanguageModelParam {
// Adding such a word to dictonaries would interfere with entering in-dictionary words. For // Adding such a word to dictonaries would interfere with entering in-dictionary words. For
// example, adding "mot" to dictionaries might interfere with entering "not". // example, adding "mot" to dictionaries might interfere with entering "not".
// This kind of OOV should be filtered out. // This kind of OOV should be filtered out.
if (distracterFilter.isDistracterToWordsInDictionaries(prevWord, targetWord)) { if (distracterFilter.isDistracterToWordsInDictionaries(prevWord, targetWord, locale)) {
return null; return null;
} }
return createAndGetLanguageModelParamOfWord(prevWord, targetWord, timestamp, return createAndGetLanguageModelParamOfWord(prevWord, targetWord, timestamp,