#7981 Removed Firefox 2 specific code and CSS

4 files changed, 9 insertions, 69 deletions

diff --git a/src/com/vaadin/terminal/gwt/client/BrowserInfo.java b/src/com/vaadin/terminal/gwt/client/BrowserInfo.java
index d10ddd4368..2229cf32f7 100644
--- a/src/com/vaadin/terminal/gwt/client/BrowserInfo.java
+++ b/src/com/vaadin/terminal/gwt/client/BrowserInfo.java
@@ -212,12 +212,6 @@ public class BrowserInfo {
         return browserDetails.isWebKit();
     }
 
-    public boolean isFF2() {
-        // FIXME: Should use browserVersion
-        return browserDetails.isFirefox()
-                && browserDetails.getBrowserEngineVersion() == 1.8;
-    }
-
     public boolean isFF3() {
         // FIXME: Should use browserVersion
         return browserDetails.isFirefox()
diff --git a/src/com/vaadin/terminal/gwt/client/ui/VPanel.java b/src/com/vaadin/terminal/gwt/client/ui/VPanel.java
index 0fcdb66568..20cfdd7ad4 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VPanel.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VPanel.java
@@ -322,11 +322,10 @@ public class VPanel extends SimplePanel implements Container,
     }
 
     public void runHacks(boolean runGeckoFix) {
-        if ((BrowserInfo.get().isIE() || BrowserInfo.get().isFF2())
-                && (width == null || width.equals(""))) {
+        if ((BrowserInfo.get().isIE()) && (width == null || width.equals(""))) {
             /*
-             * IE and FF2 needs width to be specified for the root DIV so we
-             * calculate that from the sizes of the caption and layout
+             * IE (what version??) needs width to be specified for the root DIV
+             * so we calculate that from the sizes of the caption and layout
              */
             int captionWidth = captionText.getOffsetWidth()
                     + getCaptionMarginLeft() + getCaptionPaddingHorizontal();
diff --git a/src/com/vaadin/terminal/gwt/client/ui/VWindow.java b/src/com/vaadin/terminal/gwt/client/ui/VWindow.java
index cec6bf4581..a80982ea5e 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VWindow.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VWindow.java
@@ -668,7 +668,6 @@ public class VWindow extends VOverlay implements Container,
         }
         super.show();
 
-        setFF2CaretFixEnabled(true);
         fixFF3OverflowBug();
     }
 
@@ -683,28 +682,6 @@ public class VWindow extends VOverlay implements Container,
         }
     }
 
-    /**
-     * Fix "missing cursor" browser bug workaround for FF2 in Windows and Linux.
-     * 
-     * Calling this method has no effect on other browsers than the ones based
-     * on Gecko 1.8
-     * 
-     * @param enable
-     */
-    private void setFF2CaretFixEnabled(boolean enable) {
-        if (BrowserInfo.get().isFF2()) {
-            if (enable) {
-                Scheduler.get().scheduleDeferred(new Command() {
-                    public void execute() {
-                        DOM.setStyleAttribute(getElement(), "overflow", "auto");
-                    }
-                });
-            } else {
-                DOM.setStyleAttribute(getElement(), "overflow", "");
-            }
-        }
-    }
-
     @Override
     public void hide() {
         if (vaadinModality) {
@@ -731,14 +708,6 @@ public class VWindow extends VOverlay implements Container,
     }
 
     private void showModalityCurtain() {
-        if (BrowserInfo.get().isFF2()) {
-            DOM.setStyleAttribute(
-                    getModalityCurtain(),
-                    "height",
-                    DOM.getElementPropertyInt(RootPanel.getBodyElement(),
-                            "offsetHeight") + "px");
-            DOM.setStyleAttribute(getModalityCurtain(), "position", "absolute");
-        }
         DOM.setStyleAttribute(getModalityCurtain(), "zIndex",
                 "" + (windowOrder.indexOf(this) + Z_INDEX));
         if (isShowing()) {
@@ -760,8 +729,6 @@ public class VWindow extends VOverlay implements Container,
     private void showDraggingCurtain(boolean show) {
         if (show && draggingCurtain == null) {
 
-            setFF2CaretFixEnabled(false); // makes FF2 slow
-
             draggingCurtain = DOM.createDiv();
             DOM.setStyleAttribute(draggingCurtain, "position", "absolute");
             DOM.setStyleAttribute(draggingCurtain, "top", "0px");
@@ -774,8 +741,6 @@ public class VWindow extends VOverlay implements Container,
             DOM.appendChild(RootPanel.getBodyElement(), draggingCurtain);
         } else if (!show && draggingCurtain != null) {
 
-            setFF2CaretFixEnabled(true); // makes FF2 slow
-
             DOM.removeChild(RootPanel.getBodyElement(), draggingCurtain);
             draggingCurtain = null;
         }
diff --git a/src/com/vaadin/terminal/gwt/client/ui/layout/ChildComponentContainer.java b/src/com/vaadin/terminal/gwt/client/ui/layout/ChildComponentContainer.java
index bcb972ca6a..ebb8bd41ea 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/layout/ChildComponentContainer.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/layout/ChildComponentContainer.java
@@ -9,7 +9,6 @@ import java.util.NoSuchElementException;
 import com.google.gwt.dom.client.DivElement;
 import com.google.gwt.dom.client.Document;
 import com.google.gwt.dom.client.Element;
-import com.google.gwt.dom.client.TableElement;
 import com.google.gwt.user.client.ui.Panel;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
@@ -78,29 +77,12 @@ public class ChildComponentContainer extends Panel {
         containerDIV = Document.get().createDivElement();
 
         widgetDIV = Document.get().createDivElement();
-        if (BrowserInfo.get().isFF2()) {
-            // Style style = widgetDIV.getStyle();
-            // FF2 chokes on some floats very easily. Measuring size escpecially
-            // becomes terribly slow
-            TableElement tableEl = Document.get().createTableElement();
-            tableEl.setInnerHTML("<tbody><tr><td><div></div></td></tr></tbody>");
-            DivElement div = (DivElement) tableEl.getFirstChildElement()
-                    .getFirstChildElement().getFirstChildElement()
-                    .getFirstChildElement();
-            tableEl.setCellPadding(0);
-            tableEl.setCellSpacing(0);
-            tableEl.setBorder(0);
-            div.getStyle().setProperty("padding", "0");
-
-            setElement(tableEl);
-            containerDIV = div;
-        } else {
-            setFloat(widgetDIV, "left");
-            setElement(containerDIV);
-            containerDIV.getStyle().setProperty("height", "0");
-            containerDIV.getStyle().setProperty("width", "0px");
-            containerDIV.getStyle().setProperty("overflow", "hidden");
-        }
+
+        setFloat(widgetDIV, "left");
+        setElement(containerDIV);
+        containerDIV.getStyle().setProperty("height", "0");
+        containerDIV.getStyle().setProperty("width", "0px");
+        containerDIV.getStyle().setProperty("overflow", "hidden");
 
         if (BrowserInfo.get().isIE()) {
             /*