am ec291473: Merge "Some cleanup and reinforcement"

* commit 'ec29147375dc9dee2b7b10d0c0e48c5c4b02bcf9':
  Some cleanup and reinforcement
main
Jean Chalard 2014-05-01 03:06:37 +00:00 committed by Android Git Automerger
commit ee0ddb52db
1 changed files with 30 additions and 19 deletions

View File

@ -50,23 +50,15 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
// to auto-correct, so we set this to the highest frequency that won't, i.e. 14. // to auto-correct, so we set this to the highest frequency that won't, i.e. 14.
private static final int USER_DICT_SHORTCUT_FREQUENCY = 14; private static final int USER_DICT_SHORTCUT_FREQUENCY = 14;
// TODO: use Words.SHORTCUT when we target JellyBean or above private static final String[] PROJECTION_QUERY_WITH_SHORTCUT = new String[] {
final static String SHORTCUT = "shortcut"; Words.WORD,
private static final String[] PROJECTION_QUERY; Words.SHORTCUT,
static { Words.FREQUENCY,
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN) { };
PROJECTION_QUERY = new String[] { private static final String[] PROJECTION_QUERY_WITHOUT_SHORTCUT = new String[] {
Words.WORD, Words.WORD,
SHORTCUT, Words.FREQUENCY,
Words.FREQUENCY, };
};
} else {
PROJECTION_QUERY = new String[] {
Words.WORD,
Words.FREQUENCY,
};
}
}
private static final String NAME = "userunigram"; private static final String NAME = "userunigram";
@ -179,10 +171,29 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
} else { } else {
requestArguments = localeElements; requestArguments = localeElements;
} }
final String requestString = request.toString();
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN) {
try {
addWordsFromProjectionLocked(PROJECTION_QUERY_WITH_SHORTCUT, requestString,
requestArguments);
} catch (IllegalArgumentException e) {
// This may happen on some non-compliant devices where the declared API is JB+ but
// the SHORTCUT column is not present for some reason.
addWordsFromProjectionLocked(PROJECTION_QUERY_WITHOUT_SHORTCUT, requestString,
requestArguments);
}
} else {
addWordsFromProjectionLocked(PROJECTION_QUERY_WITHOUT_SHORTCUT, requestString,
requestArguments);
}
}
private void addWordsFromProjectionLocked(final String[] query, String request,
final String[] requestArguments) throws IllegalArgumentException {
Cursor cursor = null; Cursor cursor = null;
try { try {
cursor = mContext.getContentResolver().query( cursor = mContext.getContentResolver().query(
Words.CONTENT_URI, PROJECTION_QUERY, request.toString(), requestArguments, null); Words.CONTENT_URI, query, request, requestArguments, null);
addWordsLocked(cursor); addWordsLocked(cursor);
} catch (final SQLiteException e) { } catch (final SQLiteException e) {
Log.e(TAG, "SQLiteException in the remote User dictionary process.", e); Log.e(TAG, "SQLiteException in the remote User dictionary process.", e);
@ -239,7 +250,7 @@ public class UserBinaryDictionary extends ExpandableBinaryDictionary {
if (cursor == null) return; if (cursor == null) return;
if (cursor.moveToFirst()) { if (cursor.moveToFirst()) {
final int indexWord = cursor.getColumnIndex(Words.WORD); final int indexWord = cursor.getColumnIndex(Words.WORD);
final int indexShortcut = hasShortcutColumn ? cursor.getColumnIndex(SHORTCUT) : 0; final int indexShortcut = hasShortcutColumn ? cursor.getColumnIndex(Words.SHORTCUT) : 0;
final int indexFrequency = cursor.getColumnIndex(Words.FREQUENCY); final int indexFrequency = cursor.getColumnIndex(Words.FREQUENCY);
while (!cursor.isAfterLast()) { while (!cursor.isAfterLast()) {
final String word = cursor.getString(indexWord); final String word = cursor.getString(indexWord);