From 04905e1e2eb81770d813abda6a516254e541443d Mon Sep 17 00:00:00 2001 From: Ahmed Ashour Date: Tue, 31 Oct 2017 18:51:22 +0100 Subject: Use local variable for getWidget(). (#10253) In some places, getWidget() is called many times, and I believe this can have a litte performance improvement, since in some cases super.getWidget() is called more than once. --- .../widgetset/client/ResizeTerrorizerControlConnector.java | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'uitest/src/main/java/com') diff --git a/uitest/src/main/java/com/vaadin/tests/widgetset/client/ResizeTerrorizerControlConnector.java b/uitest/src/main/java/com/vaadin/tests/widgetset/client/ResizeTerrorizerControlConnector.java index 4a4bb58488..068896f323 100644 --- a/uitest/src/main/java/com/vaadin/tests/widgetset/client/ResizeTerrorizerControlConnector.java +++ b/uitest/src/main/java/com/vaadin/tests/widgetset/client/ResizeTerrorizerControlConnector.java @@ -244,17 +244,18 @@ public class ResizeTerrorizerControlConnector extends AbstractComponentConnector @Override public void postLayout() { - if (getWidget().startWidth.getValue() == null) { + ResizeTerrorizerControlPanel panel = getWidget(); + if (panel.startWidth.getValue() == null) { int width = getTarget().getWidget().getElement().getOffsetWidth(); - getWidget().startWidth.setValue(width); - getWidget().endWidth + panel.startWidth.setValue(width); + panel.endWidth .setValue(width + getState().defaultWidthOffset); } - if (getWidget().startHeight.getValue() == null) { + if (panel.startHeight.getValue() == null) { int height = getTarget().getWidget().getElement().getOffsetHeight(); - getWidget().startHeight.setValue(height); - getWidget().endHeight + panel.startHeight.setValue(height); + panel.endHeight .setValue(height + getState().defaultHeightOffset); } } -- cgit v1.2.3