Add debug message to detect unusual zero width condition
Bug: 8452013 Change-Id: I0169dc6b0d08d1b4d9e01a5fa2ad01e9db292bdamain
parent
f67a388197
commit
bd8e92d843
|
@ -291,10 +291,22 @@ public final class MoreKeysKeyboard extends Keyboard {
|
||||||
// adjusted with their bottom paddings deducted.
|
// adjusted with their bottom paddings deducted.
|
||||||
width = keyPreviewDrawParams.mPreviewVisibleWidth;
|
width = keyPreviewDrawParams.mPreviewVisibleWidth;
|
||||||
height = keyPreviewDrawParams.mPreviewVisibleHeight + mParams.mVerticalGap;
|
height = keyPreviewDrawParams.mPreviewVisibleHeight + mParams.mVerticalGap;
|
||||||
|
// TODO: Remove this check.
|
||||||
|
if (width == 0) {
|
||||||
|
throw new IllegalArgumentException(
|
||||||
|
"Zero width key detected: " + parentKey + " in " + parentKeyboard.mId);
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
width = getMaxKeyWidth(parentKeyboardView, parentKey, mParams.mDefaultKeyWidth,
|
width = getMaxKeyWidth(parentKeyboardView, parentKey, mParams.mDefaultKeyWidth,
|
||||||
context.getResources());
|
context.getResources());
|
||||||
height = parentKeyboard.mMostCommonKeyHeight;
|
height = parentKeyboard.mMostCommonKeyHeight;
|
||||||
|
// TODO: Remove this check.
|
||||||
|
if (width == 0) {
|
||||||
|
throw new IllegalArgumentException(
|
||||||
|
"Zero width calculated: " + parentKey
|
||||||
|
+ " moreKeys=" + java.util.Arrays.toString(parentKey.mMoreKeys)
|
||||||
|
+ " in " + parentKeyboard.mId);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
final int dividerWidth;
|
final int dividerWidth;
|
||||||
if (parentKey.needsDividersInMoreKeys()) {
|
if (parentKey.needsDividersInMoreKeys()) {
|
||||||
|
|
Loading…
Reference in New Issue