BugFix for ONOS-4942.

- Was excluding EdgePort when trying to remove EdgePort...

Change-Id: I79da723da5b045e322ec9b9cdb5457a17f1aceb4

@@ -51,7 +51,6 @@ import java.nio.ByteBuffer;
 import java.util.Map;
 import java.util.Map;
 import java.util.Optional;
 import java.util.Optional;
 import java.util.Set;
 import java.util.Set;
-
 import static org.onosproject.net.device.DeviceEvent.Type.*;
 import static org.onosproject.net.device.DeviceEvent.Type.*;
 import static org.onosproject.net.edge.EdgePortEvent.Type.EDGE_PORT_ADDED;
 import static org.onosproject.net.edge.EdgePortEvent.Type.EDGE_PORT_ADDED;
 import static org.onosproject.net.edge.EdgePortEvent.Type.EDGE_PORT_REMOVED;
 import static org.onosproject.net.edge.EdgePortEvent.Type.EDGE_PORT_REMOVED;
@@ -153,6 +152,8 @@ public class EdgeManager
 
 
         @Override
         @Override
         public void event(TopologyEvent event) {
         public void event(TopologyEvent event) {
+            log.trace("Processing TopologyEvent {} caused by {}",
+                      event.subject(), event.reasons());
             topology = event.subject();
             topology = event.subject();
             event.reasons().forEach(reason -> {
             event.reasons().forEach(reason -> {
                 if (reason instanceof DeviceEvent) {
                 if (reason instanceof DeviceEvent) {
@@ -239,7 +240,8 @@ public class EdgeManager
 
 
     // Removes the specified connection point from the edge points.
     // Removes the specified connection point from the edge points.
     private void removeEdgePort(ConnectPoint point) {
     private void removeEdgePort(ConnectPoint point) {
-        if (isEdgePort(point)) {
+        // trying to remove edge ports, so we shouldn't check if it's EdgePoint
+        if (!point.port().isLogical()) {
             Set<ConnectPoint> set = connectionPoints.get(point.deviceId());
             Set<ConnectPoint> set = connectionPoints.get(point.deviceId());
             if (set == null) {
             if (set == null) {
                 return;
                 return;
@@ -250,7 +252,7 @@ public class EdgeManager
             if (set.isEmpty()) {
             if (set.isEmpty()) {
                 connectionPoints.computeIfPresent(point.deviceId(), (k, v) -> {
                 connectionPoints.computeIfPresent(point.deviceId(), (k, v) -> {
                     if (v.isEmpty()) {
                     if (v.isEmpty()) {
-                        return v;
+                        return null;
                     } else {
                     } else {
                         return v;
                         return v;
                     }
                     }