commit 86d086316912865c30d43d9120f3b9572d1bd91f
author Andrea Campanella <andrea@onlab.us> Mon Dec 07 16:27:59 2015 -0800
committer Gerrit Code Review <gerrit@onlab.us> Tue Dec 08 06:35:33 2015 +0000
tree 48b9b7f5d3e6350352edc5825e2e5e41d9ba5a41
parent dab0974ecae50eb3400572064f21d9351070478e

ONOS-3182 Fix highlight of updated paths through though delay

Change-Id: I3d33ed6340aa75ed8c31b7440c5cba81131b7168

apps/pathpainter/src/main/java/org/onosproject/pathpainter/PathPainterTopovMessageHandler.java

diff --git a/apps/pathpainter/src/main/java/org/onosproject/pathpainter/PathPainterTopovMessageHandler.java b/apps/pathpainter/src/main/java/org/onosproject/pathpainter/PathPainterTopovMessageHandler.java
index cf395e4..2e8bced 100644
--- a/apps/pathpainter/src/main/java/org/onosproject/pathpainter/PathPainterTopovMessageHandler.java
+++ b/apps/pathpainter/src/main/java/org/onosproject/pathpainter/PathPainterTopovMessageHandler.java
@@ -49,7 +49,7 @@
 import java.util.Set;
 
 /**
- * Skeletal ONOS UI Topology-Overlay message handler.
+ * ONOS UI PathPainter Topology-Overlay message handler.
  */
 public class PathPainterTopovMessageHandler extends UiMessageHandler {
 
@@ -69,12 +69,16 @@
     private static final String ENDSTATION = "endstation";
     public static final String DST = "Dst";
     public static final String SRC = "Src";
-    private static LinkWeight linkData;
+    // Delay for showHighlights event processing on GUI client side to
+    // account for addLink animation.
+    public static final int DELAY_MS = 1100;
 
     private final TopologyListener topologyListener = new InternalTopologyListener();
 
     private Set<Link> allPathLinks;
     private boolean listenersRemoved;
+    private LinkWeight linkData;
+    private int highlightDelay;
 
     private enum Mode {
         SHORTEST, DISJOINT, GEODATA, SRLG, INVALID
@@ -204,7 +208,6 @@
     }
 
 
-
     private final class NextPathHandler extends RequestHandler {
         public NextPathHandler() {
             super(PAINTER_NEXT_PATH);
@@ -326,6 +329,9 @@
                     ImmutableSet.of() : ImmutableSet.copyOf(paths.get(pathIndex).links());
         }
         Highlights highlights = new Highlights();
+        if (highlightDelay > 0) {
+            highlights.delay(highlightDelay);
+        }
         for (PathLink plink : linkMap.biLinks()) {
             plink.computeHilight(selectedPathLinks, allPathLinks);
             highlights.add(plink.highlight(null));
@@ -370,6 +376,7 @@
         listenersRemoved = false;
         topologyService.addListener(topologyListener);
     }
+
     private synchronized void removeListeners() {
         if (!listenersRemoved) {
             listenersRemoved = true;
@@ -381,7 +388,9 @@
     private class InternalTopologyListener implements TopologyListener {
         @Override
         public void event(TopologyEvent event) {
+            highlightDelay = DELAY_MS;
             findAndSendPaths(currentMode);
+            highlightDelay = 0;
         }
     }