Bug 1017242 - Part B: remove API 8 code from toolbar. r=mfinkle
authorRichard Newman <rnewman@mozilla.com>
Fri, 18 Jul 2014 16:47:10 -0700
changeset 194957 6d78d2be19f674c51d46096c3dafbde3137ed9d7
parent 194956 8bb625db40152d1f5d61aa54ade10862e98c9115
child 194958 f0fb5192286826f0f10a97e356c139ba3ca898c9
push id7814
push userrnewman@mozilla.com
push dateFri, 18 Jul 2014 23:47:24 +0000
treeherderfx-team@537990bf7786 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle
bugs1017242
milestone33.0a1
Bug 1017242 - Part B: remove API 8 code from toolbar. r=mfinkle
mobile/android/base/toolbar/ToolbarEditText.java
--- a/mobile/android/base/toolbar/ToolbarEditText.java
+++ b/mobile/android/base/toolbar/ToolbarEditText.java
@@ -387,19 +387,17 @@ public class ToolbarEditText extends Cus
                 // i.e. when the composing text is getting shorter.
                 if (composingStart >= 0 &&
                     composingEnd >= 0 &&
                     (composingEnd - composingStart) > text.length() &&
                     removeAutocomplete(editable)) {
                     // Make the IME aware that we interrupted the setComposingText call,
                     // by having finishComposingText() send change notifications to the IME.
                     finishComposingText();
-                    if (Build.VERSION.SDK_INT >= 9) {
-                        setComposingRegion(composingStart, composingEnd);
-                    }
+                    setComposingRegion(composingStart, composingEnd);
                     return true;
                 }
                 return false;
             }
 
             @Override
             public boolean commitText(CharSequence text, int newCursorPosition) {
                 if (removeAutocompleteOnComposing(text)) {