Rolling back r7635 because this is causing unexpected failures on Firefox,
which I will look at tomorrow morning.
Review at http://gwt-code-reviews.appspot.com/153815
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@7639 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/user/src/com/google/gwt/user/client/ui/FocusWidget.java b/user/src/com/google/gwt/user/client/ui/FocusWidget.java
index 4f473f7..f7390f8 100644
--- a/user/src/com/google/gwt/user/client/ui/FocusWidget.java
+++ b/user/src/com/google/gwt/user/client/ui/FocusWidget.java
@@ -272,8 +272,7 @@
// any calls made to FocusWidget.setTabIndex(int) by user code, because
// FocusWidget.setTabIndex(int) cannot be called until setElement(elem)
// has been called.
- if (-1 == getTabIndex()) {
- setTabIndex(0);
- }
+ setTabIndex(0);
}
+
}
diff --git a/user/test/com/google/gwt/user/client/ui/TextBoxTest.java b/user/test/com/google/gwt/user/client/ui/TextBoxTest.java
index 80b5f17..42a6963 100644
--- a/user/test/com/google/gwt/user/client/ui/TextBoxTest.java
+++ b/user/test/com/google/gwt/user/client/ui/TextBoxTest.java
@@ -15,9 +15,6 @@
*/
package com.google.gwt.user.client.ui;
-import com.google.gwt.dom.client.DivElement;
-import com.google.gwt.dom.client.Document;
-
/**
* Testing TextBox.
*/
@@ -35,7 +32,7 @@
// As our setText does not honor max length, no way to text it in the wild
// here.
}
-
+
public void testMinLength() {
TextBox b = createTextBoxBase();
b.setVisibleLength(5);
@@ -47,14 +44,4 @@
// Now check visible length.
assertEquals(5, b.getVisibleLength());
}
-
- public void testNoNukeTabIndex() {
- Document doc = Document.get();
- DivElement div = doc.createDivElement();
- div.setInnerHTML("<input type='text' id='tb' tabindex='1'></input>");
- doc.getBody().appendChild(div);
-
- TextBox tb = TextBox.wrap(doc.getElementById("tb"));
- assertEquals(1, tb.getTabIndex());
- }
}
