Remove unused methods. (#10058)

16 files changed, 5 insertions, 95 deletions

diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VFilterSelect.java b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VFilterSelect.java
index b78ad3928b..74c88d9c52 100644
--- a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VFilterSelect.java
+++ b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VFilterSelect.java
@@ -659,20 +659,6 @@ public class VFilterSelect extends Composite
             }
         }
 
-        private void scroll(double deltaY) {
-            boolean scrollActive = menu.isScrollActive();
-
-            debug("VFS.SP: scroll() scrollActive: " + scrollActive);
-
-            if (!scrollActive) {
-                if (deltaY > 0d) {
-                    lazyPageScroller.scrollDown();
-                } else {
-                    lazyPageScroller.scrollUp();
-                }
-            }
-        }
-
         @Override
         public void onBrowserEvent(Event event) {
             debug("VFS.SP: onBrowserEvent()");
diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VScrollTable.java b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VScrollTable.java
index 04bb24f0b9..f0f05ff476 100644
--- a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VScrollTable.java
+++ b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VScrollTable.java
@@ -6648,18 +6648,6 @@ public class VScrollTable extends FlowPanel
                 return paintableId;
             }
 
-            private int getColIndexOf(Widget child) {
-                com.google.gwt.dom.client.Element widgetCell = child
-                        .getElement().getParentElement().getParentElement();
-                NodeList<TableCellElement> cells = rowElement.getCells();
-                for (int i = 0; i < cells.getLength(); i++) {
-                    if (cells.getItem(i) == widgetCell) {
-                        return i;
-                    }
-                }
-                return -1;
-            }
-
             public Widget getWidgetForPaintable() {
                 return this;
             }
diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VTreeTable.java b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VTreeTable.java
index db691b3785..903b611ff3 100644
--- a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VTreeTable.java
+++ b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/VTreeTable.java
@@ -271,16 +271,6 @@ public class VTreeTable extends VScrollTable {
                 }
             }
 
-            private int getHierarchyAndIconWidth() {
-                int consumedSpace = treeSpacer.getOffsetWidth();
-                if (treeSpacer.getParentElement().getChildCount() > 2) {
-                    // icon next to tree spacer
-                    consumedSpace += ((com.google.gwt.dom.client.Element) treeSpacer
-                            .getNextSibling()).getOffsetWidth();
-                }
-                return consumedSpace;
-            }
-
             @Override
             protected void setCellWidth(int cellIx, int width) {
                 if (cellIx == getHierarchyColumnIndex()) {
diff --git a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/calendar/schedule/DateCellDayEvent.java b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/calendar/schedule/DateCellDayEvent.java
index 65b49b8434..7ba6d3f0a2 100644
--- a/compatibility-client/src/main/java/com/vaadin/v7/client/ui/calendar/schedule/DateCellDayEvent.java
+++ b/compatibility-client/src/main/java/com/vaadin/v7/client/ui/calendar/schedule/DateCellDayEvent.java
@@ -565,27 +565,6 @@ public class DateCellDayEvent extends FocusableHTML
         return DateConstants.MINUTEINMILLIS * 30;
     }
 
-    /**
-     * Build the string for sending resize events to server
-     *
-     * @param event
-     * @return
-     */
-    private String buildResizeString(CalendarEvent event) {
-        StringBuilder buffer = new StringBuilder();
-        buffer.append(event.getIndex());
-        buffer.append(",");
-        buffer.append(DateUtil.formatClientSideDate(event.getStart()));
-        buffer.append("-");
-        buffer.append(DateUtil.formatClientSideTime(event.getStartTime()));
-        buffer.append(",");
-        buffer.append(DateUtil.formatClientSideDate(event.getEnd()));
-        buffer.append("-");
-        buffer.append(DateUtil.formatClientSideTime(event.getEndTime()));
-
-        return buffer.toString();
-    }
-
     private Date getTargetDateByCurrentPosition(int left) {
         DateCell newParent = (DateCell) weekGrid.content
                 .getWidget((left / getDateCellWidth()) + 1);
diff --git a/compatibility-server/src/main/java/com/vaadin/v7/data/util/AbstractProperty.java b/compatibility-server/src/main/java/com/vaadin/v7/data/util/AbstractProperty.java
index 98af2cdf17..6a3c26a578 100644
--- a/compatibility-server/src/main/java/com/vaadin/v7/data/util/AbstractProperty.java
+++ b/compatibility-server/src/main/java/com/vaadin/v7/data/util/AbstractProperty.java
@@ -272,8 +272,4 @@ public abstract class AbstractProperty<T> implements Property<T>,
 
         return Collections.EMPTY_LIST;
     }
-
-    private static Logger getLogger() {
-        return Logger.getLogger(AbstractProperty.class.getName());
-    }
 }
diff --git a/compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/ColumnProperty.java b/compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/ColumnProperty.java
index b1fc3738ed..6edf06edd3 100644
--- a/compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/ColumnProperty.java
+++ b/compatibility-server/src/main/java/com/vaadin/v7/data/util/sqlcontainer/ColumnProperty.java
@@ -257,10 +257,6 @@ public final class ColumnProperty implements Property {
 
     // LegacyPropertyHelper has been removed in Vaadin 8
 
-    private static Logger getLogger() {
-        return Logger.getLogger(ColumnProperty.class.getName());
-    }
-
     public void setOwner(RowItem owner) {
         if (owner == null) {
             throw new IllegalArgumentException("Owner can not be set to null.");
diff --git a/compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractField.java b/compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractField.java
index 99bc5a1f77..4eb44994aa 100644
--- a/compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractField.java
+++ b/compatibility-server/src/main/java/com/vaadin/v7/ui/AbstractField.java
@@ -1774,8 +1774,4 @@ public abstract class AbstractField<T> extends AbstractLegacyComponent
                 super.isReadOnly(), def.isReadOnly(), Boolean.class,
                 designContext);
     }
-
-    private static final Logger getLogger() {
-        return Logger.getLogger(AbstractField.class.getName());
-    }
 }
diff --git a/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java b/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java
index fdd93f56c4..c4e7c42107 100644
--- a/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java
+++ b/compatibility-server/src/main/java/com/vaadin/v7/ui/Grid.java
@@ -1080,18 +1080,6 @@ public class Grid extends AbstractComponent
                         new ConnectorErrorEvent(Grid.this, event.getCause()));
             }
         }
-
-        private Object getFirstPropertyId(FieldGroup fieldGroup,
-                Set<Field<?>> keySet) {
-            for (Column c : getColumns()) {
-                Object propertyId = c.getPropertyId();
-                Field<?> f = fieldGroup.getField(propertyId);
-                if (keySet.contains(f)) {
-                    return propertyId;
-                }
-            }
-            return null;
-        }
     }
 
     /**
diff --git a/compatibility-server/src/test/java/com/vaadin/tests/server/SerializationTest.java b/compatibility-server/src/test/java/com/vaadin/tests/server/SerializationTest.java
index eb852974fc..b59d7a4aa7 100644
--- a/compatibility-server/src/test/java/com/vaadin/tests/server/SerializationTest.java
+++ b/compatibility-server/src/test/java/com/vaadin/tests/server/SerializationTest.java
@@ -54,9 +54,4 @@ public class SerializationTest {
 
         return s2;
     }
-
-    private static String toString(Object o) {
-        return String.valueOf(o);
-    }
-
 }
diff --git a/server/src/test/java/com/vaadin/data/BeanBinderTest.java b/server/src/test/java/com/vaadin/data/BeanBinderTest.java
index 0b67097d54..35722405f7 100644
--- a/server/src/test/java/com/vaadin/data/BeanBinderTest.java
+++ b/server/src/test/java/com/vaadin/data/BeanBinderTest.java
@@ -25,6 +25,7 @@ import com.vaadin.tests.data.bean.BeanToValidate;
 import com.vaadin.ui.CheckBoxGroup;
 import com.vaadin.ui.TextField;
 
+@SuppressWarnings("unused")
 public class BeanBinderTest
         extends BinderTestBase<Binder<BeanToValidate>, BeanToValidate> {
 
diff --git a/server/src/test/java/com/vaadin/data/BinderBookOfVaadinTest.java b/server/src/test/java/com/vaadin/data/BinderBookOfVaadinTest.java
index e0b8330ccc..89987d5f62 100644
--- a/server/src/test/java/com/vaadin/data/BinderBookOfVaadinTest.java
+++ b/server/src/test/java/com/vaadin/data/BinderBookOfVaadinTest.java
@@ -46,6 +46,7 @@ import com.vaadin.ui.TextField;
  * @author Vaadin Ltd
  *
  */
+@SuppressWarnings("unused")
 public class BinderBookOfVaadinTest {
 
     private static class BookPerson {
diff --git a/server/src/test/java/com/vaadin/util/ReflectToolsGetFieldValueByTypeTest.java b/server/src/test/java/com/vaadin/util/ReflectToolsGetFieldValueByTypeTest.java
index ebba16e0d0..1ce40b8c9d 100644
--- a/server/src/test/java/com/vaadin/util/ReflectToolsGetFieldValueByTypeTest.java
+++ b/server/src/test/java/com/vaadin/util/ReflectToolsGetFieldValueByTypeTest.java
@@ -7,6 +7,7 @@ import java.lang.reflect.Field;
 
 import org.junit.Test;
 
+@SuppressWarnings("unused")
 public class ReflectToolsGetFieldValueByTypeTest {
     @Test
     public void getFieldValue() {
diff --git a/uitest/src/main/java/com/vaadin/tests/layouts/CaptionsInLayoutsWaiAria.java b/uitest/src/main/java/com/vaadin/tests/layouts/CaptionsInLayoutsWaiAria.java
index 9bcb9cd799..38f6da9a21 100644
--- a/uitest/src/main/java/com/vaadin/tests/layouts/CaptionsInLayoutsWaiAria.java
+++ b/uitest/src/main/java/com/vaadin/tests/layouts/CaptionsInLayoutsWaiAria.java
@@ -178,6 +178,7 @@ public class CaptionsInLayoutsWaiAria extends TestBase {
         }
     }
 
+    @SuppressWarnings("unused")
     private Component toggleError() {
         CheckBox errorToggle = new CheckBox();
         errorToggle.setCaption("Error");
diff --git a/uitest/src/main/java/com/vaadin/tests/minitutorials/v7_4/ConfiguringColumnWidths.java b/uitest/src/main/java/com/vaadin/tests/minitutorials/v7_4/ConfiguringColumnWidths.java
index 5e62bc1568..8827684724 100644
--- a/uitest/src/main/java/com/vaadin/tests/minitutorials/v7_4/ConfiguringColumnWidths.java
+++ b/uitest/src/main/java/com/vaadin/tests/minitutorials/v7_4/ConfiguringColumnWidths.java
@@ -20,6 +20,7 @@ import com.vaadin.ui.UI;
 import com.vaadin.v7.ui.Grid;
 import com.vaadin.v7.ui.Grid.SelectionMode;
 
+@SuppressWarnings("unused")
 public class ConfiguringColumnWidths extends UI {
 
     @Override
diff --git a/uitest/src/main/java/com/vaadin/tests/minitutorials/v7a1/UsingBeanValidation.java b/uitest/src/main/java/com/vaadin/tests/minitutorials/v7a1/UsingBeanValidation.java
index ccef80e4cb..2e05ee9f39 100644
--- a/uitest/src/main/java/com/vaadin/tests/minitutorials/v7a1/UsingBeanValidation.java
+++ b/uitest/src/main/java/com/vaadin/tests/minitutorials/v7a1/UsingBeanValidation.java
@@ -62,11 +62,6 @@ public class UsingBeanValidation extends UI {
         public int getAge() {
             return age;
         }
-
-        private void setN() {
-            // TODO Auto-generated method stub
-
-        }
     }
 
     @Override
diff --git a/uitest/src/test/java/com/vaadin/tests/tooltip/TooltipInWindowTest.java b/uitest/src/test/java/com/vaadin/tests/tooltip/TooltipInWindowTest.java
index b005fd07a1..771f7528af 100644
--- a/uitest/src/test/java/com/vaadin/tests/tooltip/TooltipInWindowTest.java
+++ b/uitest/src/test/java/com/vaadin/tests/tooltip/TooltipInWindowTest.java
@@ -45,10 +45,6 @@ public class TooltipInWindowTest extends TooltipTest {
         clearTooltip();
     }
 
-    private WebElement getTooltipContainerElement() {
-        return getDriver().findElement(By.className("v-tooltip"));
-    }
-
     private void ensureVisibleTooltipPositionedCorrectly(WebElement textfield)
             throws InterruptedException {
         int tooltipX = getTooltip().getLocation().getX();