GUI -- TopoView - Implemented show/hide summary 'O' keystroke.

Change-Id: Id5014614b8faec19423aa367ad3de1ca3140b882

@@ -28,7 +28,7 @@
     ];
     ];
 
 
     // references to injected services etc.
     // references to injected services etc.
-    var $log, fs, ks, zs, gs, ms, sus, tfs, tis;
+    var $log, fs, ks, zs, gs, ms, sus, tes, tfs, tps, tis;
 
 
     // DOM elements
     // DOM elements
     var ovtopo, svg, defs, zoomLayer, mapG, forceG, noDevsLayer;
     var ovtopo, svg, defs, zoomLayer, mapG, forceG, noDevsLayer;
@@ -42,7 +42,7 @@
         // key bindings need to be made after the services have been injected
         // key bindings need to be made after the services have been injected
         // thus, deferred to here...
         // thus, deferred to here...
         ks.keyBindings({
         ks.keyBindings({
-            //O: [toggleSummary, 'Toggle ONOS summary pane'],
+            O: [toggleSummary, 'Toggle ONOS summary pane'],
             I: [toggleInstances, 'Toggle ONOS instances pane'],
             I: [toggleInstances, 'Toggle ONOS instances pane'],
             //D: [toggleDetails, 'Disable / enable details pane'],
             //D: [toggleDetails, 'Disable / enable details pane'],
 
 
@@ -87,6 +87,15 @@
 
 
     // --- Keystroke functions -------------------------------------------
     // --- Keystroke functions -------------------------------------------
 
 
+    function toggleSummary() {
+        if (tps.summaryVisible()) {
+            tes.sendEvent("cancelSummary");
+            tps.hideSummaryPanel();
+        } else {
+            tes.sendEvent('requestSummary');
+        }
+    }
+
     function toggleInstances() {
     function toggleInstances() {
         tis.toggle();
         tis.toggle();
         tfs.updateDeviceColors();
         tfs.updateDeviceColors();
@@ -213,7 +222,7 @@
             'TopoInstService',
             'TopoInstService',
 
 
         function ($scope, _$log_, $loc, $timeout, _fs_, mast,
         function ($scope, _$log_, $loc, $timeout, _fs_, mast,
-                  _ks_, _zs_, _gs_, _ms_, _sus_, tes, _tfs_, tps, _tis_) {
+                  _ks_, _zs_, _gs_, _ms_, _sus_, _tes_, _tfs_, _tps_, _tis_) {
             var self = this,
             var self = this,
                 projection,
                 projection,
                 dim,
                 dim,
@@ -221,7 +230,7 @@
                     // provides function calls back into this space
                     // provides function calls back into this space
                     showNoDevs: showNoDevs,
                     showNoDevs: showNoDevs,
                     projection: function () { return projection; },
                     projection: function () { return projection; },
-                    sendEvent: tes.sendEvent
+                    sendEvent: _tes_.sendEvent
                 };
                 };
 
 
             $log = _$log_;
             $log = _$log_;
@@ -231,7 +240,9 @@
             gs = _gs_;
             gs = _gs_;
             ms = _ms_;
             ms = _ms_;
             sus = _sus_;
             sus = _sus_;
+            tes = _tes_;
             tfs = _tfs_;
             tfs = _tfs_;
+            tps = _tps_;
             tis = _tis_;
             tis = _tis_;
 
 
             self.notifyResize = function () {
             self.notifyResize = function () {

@@ -58,14 +58,15 @@
         };
         };
     }
     }
 
 
-    var dispatcher = {
+    var nilApi = {},
+        dispatcher = {
         handleEvent: function (ev) {
         handleEvent: function (ev) {
             var eid = ev.event,
             var eid = ev.event,
-                api = evApis[eid] || {},
+                api = evApis[eid] || nilApi,
                 eh = api[eid];
                 eh = api[eid];
 
 
             if (eh) {
             if (eh) {
-                $log.debug('  *EVENT* ', eid, ev.payload);
+                $log.debug(' << *Rx* ', eid, ev.payload);
                 eh(ev.payload);
                 eh(ev.payload);
             } else {
             } else {
                 $log.warn('Unknown event (ignored):', ev);
                 $log.warn('Unknown event (ignored):', ev);
@@ -74,6 +75,7 @@
 
 
         sendEvent: function (evType, payload) {
         sendEvent: function (evType, payload) {
             if (wsock) {
             if (wsock) {
+                $log.debug(' *Tx* >> ', evType, payload);
                 wes.sendEvent(wsock, evType, payload);
                 wes.sendEvent(wsock, evType, payload);
             } else {
             } else {
                 $log.warn('sendEvent: no websocket open:', evType, payload);
                 $log.warn('sendEvent: no websocket open:', evType, payload);

@@ -143,7 +143,12 @@
 
 
     function showSummaryPanel() {
     function showSummaryPanel() {
         summaryPanel.show();
         summaryPanel.show();
-        // TODO: augment, once we have the details pane also
+        // TODO: augment, for details panel move
+    }
+
+    function hideSummaryPanel() {
+        summaryPanel.hide();
+        // TODO: augment, for details panel move
     }
     }
 
 
     function showDetailPanel() {
     function showDetailPanel() {
@@ -194,6 +199,7 @@
                 displayMulti: displayMulti,
                 displayMulti: displayMulti,
                 addAction: addAction,
                 addAction: addAction,
 
 
+                hideSummaryPanel: hideSummaryPanel,
                 showDetailPanel: showDetailPanel,
                 showDetailPanel: showDetailPanel,
                 hideDetailPanel: hideDetailPanel,
                 hideDetailPanel: hideDetailPanel,
 
 

@@ -40,6 +40,7 @@ describe('factory: view/topo/topoPanel.js', function() {
             'displaySingle',
             'displaySingle',
             'displayMulti',
             'displayMulti',
             'addAction',
             'addAction',
+            'hideSummaryPanel',
             'showDetailPanel',
             'showDetailPanel',
             'hideDetailPanel',
             'hideDetailPanel',
             'detailVisible',
             'detailVisible',