Browse Source

Assert that connector needing layout is attached (#11698)

Change-Id: Ie2d1ec330b857497c7673f3805e35b452f409f3c
tags/7.1.0.beta1
Leif Åstrand 11 years ago
parent
commit
cb7b02d4e5
1 changed files with 13 additions and 0 deletions
  1. 13
    0
      client/src/com/vaadin/client/LayoutManager.java

+ 13
- 0
client/src/com/vaadin/client/LayoutManager.java View File

@@ -827,6 +827,7 @@ public class LayoutManager {
* the managed layout that should be layouted
*/
public final void setNeedsHorizontalLayout(ManagedLayout layout) {
assert isAttached(layout);
needsHorizontalLayout.add(layout.getConnectorId());
}

@@ -842,9 +843,21 @@ public class LayoutManager {
* the managed layout that should be layouted
*/
public final void setNeedsVerticalLayout(ManagedLayout layout) {
assert isAttached(layout);
needsVerticalLayout.add(layout.getConnectorId());
}

private boolean isAttached(ServerConnector connector) {
while (connector != null) {
connector = connector.getParent();
if (connector == connection.getUIConnector()) {
return true;
}
}
// Reaching null parent before reaching UI connector -> not attached
return false;
}

/**
* Gets the outer height (including margins, paddings and borders) of the
* given element, provided that it has been measured. These elements are

Loading…
Cancel
Save