Remove the dialog to insert words to the dictionary.

Bug: 8562529
Change-Id: Ibd853f0cb995e7fb35f8730eafcef73ead58c869
main
Jean Chalard 2013-04-12 15:58:18 +09:00
parent d0623ad7f3
commit 1a51833828
3 changed files with 63 additions and 22 deletions

View File

@ -0,0 +1,51 @@
/*
* Copyright (C) 2013 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.inputmethod.compat;
import android.content.Context;
import android.provider.UserDictionary.Words;
import java.lang.reflect.Method;
import java.util.Locale;
public final class UserDictionaryCompatUtils {
// UserDictionary.Words#addWord(Context, String, int, String, Locale) was introduced
// in API level 16 (Build.VERSION_CODES.JELLY_BEAN).
private static final Method METHOD_addWord = CompatUtils.getMethod(Words.class, "addWord",
Context.class, String.class, Integer.TYPE, String.class, Locale.class);
public static void addWord(final Context context, final String word, final int freq,
final String shortcut, final Locale locale) {
if (hasNewerAddWord()) {
CompatUtils.invoke(Words.class, null, METHOD_addWord, context, word, freq, shortcut,
locale);
} else {
// Fall back to the pre-JellyBean method.
final int localeType;
if (null == locale) {
localeType = Words.LOCALE_TYPE_ALL;
} else {
localeType = Words.LOCALE_TYPE_CURRENT;
}
Words.addWord(context, word, freq, localeType);
}
}
private static final boolean hasNewerAddWord() {
return null != METHOD_addWord;
}
}

View File

@ -1243,10 +1243,6 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
} else { } else {
wordToEdit = word; wordToEdit = word;
} }
mPositionalInfoForUserDictPendingAddition =
new PositionalInfoForUserDictPendingAddition(
wordToEdit, mLastSelectionEnd, getCurrentInputEditorInfo(),
mLastComposedWord.mCapitalizedMode);
mUserDictionary.addWordToUserDictionary(wordToEdit); mUserDictionary.addWordToUserDictionary(wordToEdit);
} }

View File

@ -20,7 +20,6 @@ import android.content.ContentProviderClient;
import android.content.ContentResolver; import android.content.ContentResolver;
import android.content.ContentUris; import android.content.ContentUris;
import android.content.Context; import android.content.Context;
import android.content.Intent;
import android.database.ContentObserver; import android.database.ContentObserver;
import android.database.Cursor; import android.database.Cursor;
import android.net.Uri; import android.net.Uri;
@ -28,7 +27,10 @@ import android.os.Build;
import android.provider.UserDictionary.Words; import android.provider.UserDictionary.Words;
import android.text.TextUtils; import android.text.TextUtils;
import com.android.inputmethod.compat.UserDictionaryCompatUtils;
import java.util.Arrays; import java.util.Arrays;
import java.util.Locale;
/** /**
* An expandable dictionary that stores the words in the user dictionary provider into a binary * An expandable dictionary that stores the words in the user dictionary provider into a binary
@ -61,10 +63,6 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
private static final String NAME = "userunigram"; private static final String NAME = "userunigram";
// This is not exported by the framework so we pretty much have to write it here verbatim
private static final String ACTION_USER_DICTIONARY_INSERT =
"com.android.settings.USER_DICTIONARY_INSERT";
private ContentObserver mObserver; private ContentObserver mObserver;
final private String mLocale; final private String mLocale;
final private boolean mAlsoUseMoreRestrictiveLocales; final private boolean mAlsoUseMoreRestrictiveLocales;
@ -211,23 +209,19 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
/** /**
* Adds a word to the user dictionary and makes it persistent. * Adds a word to the user dictionary and makes it persistent.
* *
* This will call upon the system interface to do the actual work through the intent readied by
* the system to this effect.
*
* @param word the word to add. If the word is capitalized, then the dictionary will * @param word the word to add. If the word is capitalized, then the dictionary will
* recognize it as a capitalized word when searched. * recognize it as a capitalized word when searched.
*/ */
public synchronized void addWordToUserDictionary(final String word) { public synchronized void addWordToUserDictionary(final String word) {
// TODO: do something for the UI. With the following, any sufficiently long word will // Update the user dictionary provider
// look like it will go to the user dictionary but it won't. final Locale locale;
// Safeguard against adding long words. Can cause stack overflow. if (USER_DICTIONARY_ALL_LANGUAGES == mLocale) {
if (word.length() >= MAX_WORD_LENGTH) return; locale = null;
} else {
Intent intent = new Intent(ACTION_USER_DICTIONARY_INSERT); locale = LocaleUtils.constructLocaleFromString(mLocale);
intent.putExtra(Words.WORD, word); }
intent.putExtra(Words.LOCALE, mLocale); UserDictionaryCompatUtils.addWord(mContext, word,
intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); HISTORICAL_DEFAULT_USER_DICTIONARY_FREQUENCY, null, locale);
mContext.startActivity(intent);
} }
private int scaleFrequencyFromDefaultToLatinIme(final int defaultFrequency) { private int scaleFrequencyFromDefaultToLatinIme(final int defaultFrequency) {