From: Todd Brandt Date: Mon, 27 Aug 2012 16:56:53 +0000 (-0700) Subject: checked in a small fix to the api.js to handle when the service X-Git-Tag: submit/release/20120910.223349~56 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1e39825a8f5dab7ae5009085d9c4adb6c7d15d8f;p=profile%2Fivi%2Fautomotive-message-broker.git checked in a small fix to the api.js to handle when the service doesn't return the proper acknowledgement on unsubscribe --- diff --git a/plugins/websocketsink/test/test.js b/plugins/websocketsink/test/test.js index 1d2232c..72383ee 100644 --- a/plugins/websocketsink/test/test.js +++ b/plugins/websocketsink/test/test.js @@ -176,8 +176,8 @@ function subscribe(eventlist) { var sub = data[i]+"_subscribe"; var unsub = data[i]+"_unsubscribe"; - document.getElementById(sub).className = "testbutton subscribe disable" - document.getElementById(unsub).className = "testbutton unsubscribe"; +// document.getElementById(sub).className = "testbutton subscribe disable" +// document.getElementById(unsub).className = "testbutton unsubscribe"; document.addEventListener(data[i], eventListener, false); } }, @@ -189,6 +189,12 @@ function subscribe(eventlist) function unsubscribe(eventlist) { + /* kill the handers first, so even if the service fails to acknowledge */ + /* we've stopped listening */ + for(var i = 0; i < eventlist.length; i++) + { + document.removeEventListener(eventlist[i], eventListener, false); + } window.vehicle.unsubscribe(eventlist, function(data) { PRINT.pass("Unsubscribe success for: "+data); @@ -196,9 +202,8 @@ function unsubscribe(eventlist) { var sub = data[i]+"_subscribe"; var unsub = data[i]+"_unsubscribe"; - document.getElementById(unsub).className = "testbutton unsubscribe disable"; - document.getElementById(sub).className = "testbutton subscribe"; - document.removeEventListener(data[i], eventListener, false); +// document.getElementById(unsub).className = "testbutton unsubscribe disable"; +// document.getElementById(sub).className = "testbutton subscribe"; } }, function(msg) {