Revert "Fix for 'Aborting layout after 100 passess' (#13359)"

Causes regression with IE8:
http://r2d2.devnet.vaadin.com:8111/viewLog.html?buildTypeId=Vaadin72_Vaadin72DevelopmentBuildTb2Tests&buildId=86020

Change-Id: I6d848777b28a1d3f27a25fec778cba8d68a45690

4 files changed, 23 insertions, 271 deletions

diff --git a/client/src/com/vaadin/client/LayoutManager.java b/client/src/com/vaadin/client/LayoutManager.java
index e17c0233b3..bf79009f4c 100644
--- a/client/src/com/vaadin/client/LayoutManager.java
+++ b/client/src/com/vaadin/client/LayoutManager.java
@@ -36,7 +36,6 @@ import com.vaadin.client.ui.VNotification;
 import com.vaadin.client.ui.layout.ElementResizeEvent;
 import com.vaadin.client.ui.layout.ElementResizeListener;
 import com.vaadin.client.ui.layout.LayoutDependencyTree;
-import com.vaadin.client.ui.orderedlayout.AbstractOrderedLayoutConnector;
 
 public class LayoutManager {
     private static final String LOOP_ABORT_MESSAGE = "Aborting layout after 100 passes. This would probably be an infinite loop.";
@@ -721,16 +720,6 @@ public class LayoutManager {
         Profiler.enter("LayoutManager.measureConnector");
         Element element = connector.getWidget().getElement();
         MeasuredSize measuredSize = getMeasuredSize(connector);
-        if (isBrowserOptimizedMeasuringNeeded()
-                && isWrappedInsideExpandBlock(connector)) {
-            // this fixes zoom/sub-pixel issues with ie9+ and Chrome
-            // (#13359)
-            // Update measuring logic to round up and/or down depending on
-            // browser.
-            measuredSize.setMeasurer(new MeasuredSize.RoundingMeasurer());
-        } else {
-            measuredSize.setMeasurer(null);// resets to default measurer
-        }
         MeasureResult measureResult = measuredAndUpdate(element, measuredSize);
 
         if (measureResult.isChanged()) {
@@ -740,23 +729,6 @@ public class LayoutManager {
         Profiler.leave("LayoutManager.measureConnector");
     }
 
-    /**
-     * Return true if browser may need some optimized width and height measuring
-     * for MeasuredSize.
-     * <p>
-     * Usually optimization is needed to avoid unnecessary scroll bars appearing
-     * in layouts caused by sub-pixel rounding. And to avoid LayoutManager
-     * doLayout() going into a loop trying to fit content with fractional size
-     * (percentages) inside a parent element and stopping only after safety
-     * iteration limit exceeds, also caused by sub-pixel rounding.
-     * <p>
-     * For internal use only. May be removed or replaced in the future.
-     */
-    private static boolean isBrowserOptimizedMeasuringNeeded() {
-        return BrowserInfo.get().isChrome()
-                || (BrowserInfo.get().isIE() && !BrowserInfo.get().isIE8());
-    }
-
     private void onConnectorChange(ComponentConnector connector,
             boolean widthChanged, boolean heightChanged) {
         Profiler.enter("LayoutManager.onConnectorChange");
@@ -838,26 +810,6 @@ public class LayoutManager {
         return connector instanceof ManagedLayout;
     }
 
-    /**
-     * Is the given connector wrapped inside a 'expanding' content. Detected by
-     * checking if the connector's parent is AbstractOrderedLayoutConnector that
-     * expands. 'Expand' means that some layout slots may expand its content
-     * width or height to some percentage fraction.
-     * 
-     * @param connector
-     *            The connector to check
-     * @return True if connector is wrapped inside a
-     *         AbstractOrderedLayoutConnector that expands
-     */
-    private static boolean isWrappedInsideExpandBlock(
-            ComponentConnector connector) {
-        ServerConnector parent = connector.getParent();
-        if (parent instanceof AbstractOrderedLayoutConnector) {
-            return ((AbstractOrderedLayoutConnector) parent).needsExpand();
-        }
-        return false;
-    }
-
     public void forceLayout() {
         ConnectorMap connectorMap = connection.getConnectorMap();
         JsArrayObject<ComponentConnector> componentConnectors = connectorMap
diff --git a/client/src/com/vaadin/client/MeasuredSize.java b/client/src/com/vaadin/client/MeasuredSize.java
index 4005a2651d..2531ff9389 100644
--- a/client/src/com/vaadin/client/MeasuredSize.java
+++ b/client/src/com/vaadin/client/MeasuredSize.java
@@ -43,49 +43,6 @@ public class MeasuredSize {
         }
     }
 
-    public interface Measurer {
-        int measureHeight(Element element);
-
-        int measureWidth(Element element);
-    }
-
-    /**
-     * Default measurer rounds always up.
-     */
-    public static class DefaultMeasurer implements Measurer {
-
-        @Override
-        public int measureHeight(Element element) {
-            int requiredHeight = Util.getRequiredHeight(element);
-            return requiredHeight;
-        }
-
-        @Override
-        public int measureWidth(Element element) {
-            int requiredWidth = Util.getRequiredWidth(element);
-            return requiredWidth;
-        }
-    }
-
-    /**
-     * RoundingMeasurer measurer rounds up and down, optimized for different
-     * browsers.
-     */
-    public static class RoundingMeasurer implements Measurer {
-
-        @Override
-        public int measureHeight(Element element) {
-            double requiredHeight = Util.getPreciseRequiredHeight(element);
-            return Util.roundPreciseSize(requiredHeight);
-        }
-
-        @Override
-        public int measureWidth(Element element) {
-            double requiredWidth = Util.getPreciseRequiredWidth(element);
-            return Util.roundPreciseSize(requiredWidth);
-        }
-    }
-
     private int width = -1;
     private int height = -1;
 
@@ -95,15 +52,6 @@ public class MeasuredSize {
 
     private FastStringSet dependents = FastStringSet.create();
 
-    private Measurer measurer = new DefaultMeasurer();
-
-    public void setMeasurer(Measurer measurer) {
-        if (measurer == null) {
-            measurer = new DefaultMeasurer();
-        }
-        this.measurer = measurer;
-    }
-
     public int getOuterHeight() {
         return height;
     }
@@ -288,7 +236,7 @@ public class MeasuredSize {
         Profiler.leave("Measure borders");
 
         Profiler.enter("Measure height");
-        int requiredHeight = measurer.measureHeight(element);
+        int requiredHeight = Util.getRequiredHeight(element);
         int marginHeight = sumHeights(margins);
         int oldHeight = height;
         int oldWidth = width;
@@ -299,7 +247,7 @@ public class MeasuredSize {
         Profiler.leave("Measure height");
 
         Profiler.enter("Measure width");
-        int requiredWidth = measurer.measureWidth(element);
+        int requiredWidth = Util.getRequiredWidth(element);
         int marginWidth = sumWidths(margins);
         if (setOuterWidth(requiredWidth + marginWidth)) {
             debugSizeChange(element, "Width (outer)", oldWidth, width);
diff --git a/client/src/com/vaadin/client/Util.java b/client/src/com/vaadin/client/Util.java
index bec88032bb..e031b37422 100644
--- a/client/src/com/vaadin/client/Util.java
+++ b/client/src/com/vaadin/client/Util.java
@@ -627,48 +627,16 @@ public class Util {
         return reqHeight;
     }
 
-    /**
-     * Gets the border-box width for the given element, i.e. element width +
-     * border + padding.
-     * 
-     * @param element
-     *            The element to check
-     * @return The border-box width for the element
-     */
-    public static double getPreciseRequiredWidth(
-            com.google.gwt.dom.client.Element element) {
-        double reqWidth;
-        if (browserUsesPreciseSizeByComputedStyle()) {
-            reqWidth = getPreciseWidthByComputedStyle(element);
-        } else {
-            reqWidth = getPreciseBoundingClientRectWidth(element);
-        }
-        return reqWidth;
-    }
-
-    /**
-     * Gets the border-box height for the given element, i.e. element height +
-     * border + padding.
-     * 
-     * @param element
-     *            The element to check
-     * @return The border-box height for the element
-     */
-    public static double getPreciseRequiredHeight(
-            com.google.gwt.dom.client.Element element) {
-        double reqHeight;
-        if (browserUsesPreciseSizeByComputedStyle()) {
-            reqHeight = getPreciseHeightByComputedStyle(element);
+    public static native int getRequiredWidthBoundingClientRect(
+            com.google.gwt.dom.client.Element element)
+    /*-{
+        if (element.getBoundingClientRect) {
+          var rect = element.getBoundingClientRect();
+          return Math.ceil(rect.right - rect.left);
         } else {
-            reqHeight = getPreciseBoundingClientRectHeight(element);
+          return element.offsetWidth;
         }
-        return reqHeight;
-    }
-
-    public static int getRequiredWidthBoundingClientRect(
-            com.google.gwt.dom.client.Element element) {
-        return (int) Math.ceil(getPreciseBoundingClientRectWidth(element));
-    }
+    }-*/;
 
     public static native int getRequiredHeightComputedStyle(
             com.google.gwt.dom.client.Element element)
@@ -710,10 +678,18 @@ public class Util {
          return Math.ceil(width+border+padding);
      }-*/;
 
-    public static int getRequiredHeightBoundingClientRect(
-            com.google.gwt.dom.client.Element element) {
-        return (int) Math.ceil(getPreciseBoundingClientRectHeight(element));
-    }
+    public static native int getRequiredHeightBoundingClientRect(
+            com.google.gwt.dom.client.Element element)
+    /*-{
+        var height;
+        if (element.getBoundingClientRect != null) {
+          var rect = element.getBoundingClientRect();
+          height = Math.ceil(rect.bottom - rect.top);
+        } else {
+          height = element.offsetHeight;
+        }
+        return height;
+    }-*/;
 
     public static int getRequiredWidth(Widget widget) {
         return getRequiredWidth(widget.getElement());
@@ -724,128 +700,6 @@ public class Util {
     }
 
     /**
-     * Rounds pixel size up or down depending on the browser.
-     * <p>
-     * <li>
-     * IE9, IE10 - rounds always down to closest integer
-     * <li>
-     * Others - rounds half up to closest integer (1.49 -> 1.0, 1.5 -> 2.0)
-     * 
-     * @param preciseSize
-     *            Decimal number to round.
-     * @return Rounded integer
-     */
-    public static int roundPreciseSize(double preciseSize) {
-        boolean roundAlwaysDown = BrowserInfo.get().isIE9()
-                || BrowserInfo.get().isIE10();
-        if (roundAlwaysDown) {
-            return (int) preciseSize;
-        } else {
-            // round to closest integer
-            return (int) Math.round(preciseSize);
-        }
-    }
-
-    /**
-     * Returns getBoundingClientRect.width, if supported. Otherwise return
-     * offsetWidth. Includes the padding, scrollbar, and the border. Excludes
-     * the margin.
-     * 
-     * @param elem
-     *            Target element to measure
-     */
-    private static native double getPreciseBoundingClientRectWidth(Element elem)
-    /*-{
-         try {
-             if (elem.getBoundingClientRect) {
-                 return elem.getBoundingClientRect().width;
-             } else {
-                 return (elem.offsetWidth) || 0;
-             }
-         } catch (e) {
-             // JS exception is thrown if the elem is not attached to the document.
-             return 0;
-         }
-     }-*/;
-
-    /**
-     * Returns getBoundingClientRect.height, if supported. Otherwise return
-     * offsetHeight. Includes the padding, scrollbar, and the border. Excludes
-     * the margin.
-     * 
-     * @param elem
-     *            Target element to measure
-     */
-    private static native double getPreciseBoundingClientRectHeight(Element elem)
-    /*-{
-         try {
-             if (elem.getBoundingClientRect) {
-                 return elem.getBoundingClientRect().height;
-             } else {
-                 return (elem.offsetHeight) || 0;
-             }
-         } catch (e) {
-             // JS exception is thrown if the elem is not attached to the document.
-             return 0;
-         }
-     }-*/;
-
-    /**
-     * Parse computed styles to get precise width for the given element.
-     * Excludes the margin. Fallback to offsetWidth if computed styles is not
-     * defined, or if width can't be read or it's not fractional.
-     * 
-     * @param elem
-     *            Target element to measure
-     */
-    private static native double getPreciseWidthByComputedStyle(
-            com.google.gwt.dom.client.Element elem)
-    /*-{
-        var cs = elem.ownerDocument.defaultView.getComputedStyle(elem);
-        var size = cs && cs.getPropertyValue('width') || '';
-        if (size && size.indexOf('.') > -1) {
-            size = parseFloat(size)
-                        + parseInt(cs.getPropertyValue('padding-left'))
-                        + parseInt(cs.getPropertyValue('padding-right'))
-                        + parseInt(cs.getPropertyValue('border-left-width'))
-                        + parseInt(cs.getPropertyValue('border-right-width'));
-        } else {
-            size = elem.offsetWidth;
-        }
-        return size;
-     }-*/;
-
-    /**
-     * Parse computed styles to get precise height for the given element.
-     * Excludes the margin. Fallback to offsetHeight if computed styles is not
-     * defined, or if height can't be read or it's not fractional.
-     * 
-     * @param elem
-     *            Target element to measure
-     */
-    private static native double getPreciseHeightByComputedStyle(
-            com.google.gwt.dom.client.Element elem)
-    /*-{
-        var cs = elem.ownerDocument.defaultView.getComputedStyle(elem);
-        var size = cs && cs.getPropertyValue('height') || '';
-        if (size && size.indexOf('.') > -1) {
-            size = parseFloat(size)
-                        + parseInt(cs.getPropertyValue('padding-top'))
-                        + parseInt(cs.getPropertyValue('padding-bottom'))
-                        + parseInt(cs.getPropertyValue('border-top-width'))
-                        + parseInt(cs.getPropertyValue('border-bottom-width'));
-        } else {
-            size = elem.offsetHeight;
-        }
-        return size;
-     }-*/;
-
-    /** For internal use only. May be removed or replaced in the future. */
-    private static boolean browserUsesPreciseSizeByComputedStyle() {
-        return BrowserInfo.get().isIE9();
-    }
-
-    /**
      * Detects what is currently the overflow style attribute in given element.
      * 
      * @param pe
diff --git a/client/src/com/vaadin/client/ui/orderedlayout/AbstractOrderedLayoutConnector.java b/client/src/com/vaadin/client/ui/orderedlayout/AbstractOrderedLayoutConnector.java
index c20eda71dc..0c09ae49c6 100644
--- a/client/src/com/vaadin/client/ui/orderedlayout/AbstractOrderedLayoutConnector.java
+++ b/client/src/com/vaadin/client/ui/orderedlayout/AbstractOrderedLayoutConnector.java
@@ -551,10 +551,8 @@ public abstract class AbstractOrderedLayoutConnector extends
 
     /**
      * Does the layout need to expand?
-     * <p>
-     * For internal use only. May be removed or replaced in the future.
      */
-    public boolean needsExpand() {
+    private boolean needsExpand() {
         return needsExpand;
     }