Delete duplicate call of view.rebuild. Return calling newPacket in Pl…

…atform.runLater to fix uiTests
kisel · Oct 25, 2016 · e6b8804 · e6b8804
1 parent ac31e48
commit e6b8804
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/main/java/com/xored/javafx/packeteditor/controllers/FieldEditorController.java b/src/main/java/com/xored/javafx/packeteditor/controllers/FieldEditorController.java
@@ -6,7 +6,6 @@
 import com.google.inject.name.Named;
 import com.xored.javafx.packeteditor.data.FieldEditorModel;
 import com.xored.javafx.packeteditor.data.combined.CombinedField;
-import com.xored.javafx.packeteditor.data.combined.CombinedProtocolModel;
 import com.xored.javafx.packeteditor.data.user.DocumentFile;
 import com.xored.javafx.packeteditor.events.RebuildViewEvent;
 import com.xored.javafx.packeteditor.scapy.PacketData;
@@ -77,7 +76,7 @@ public IMetadataService getMetadataService() {
     public void initialize(URL location, ResourceBundle resources) {
         view.setParentPane(fieldEditorPane);
         if (packetController.isInitialized()) {
-            this.newPacket();
+            Platform.runLater(this::newPacket);
         } else {
             view.displayConnectionError();
         }
@@ -122,11 +121,7 @@ public void handleRebuildViewEvent(RebuildViewEvent event) {
         fieldEditorTopPane.getChildren().add(snapView);
         // Rebuild content
         view.rebuild(event.getModel());
-        CombinedProtocolModel combomodel = event.getModel();
-
-        // Rebuild content
-        view.rebuild(combomodel);
-
+
         Platform.runLater(()-> {
             // Save scroll position workaround: runLater inside runLater :)
             Platform.runLater(() -> {