From: Jeongkyun Pu <jk.pu@samsung.com>
Date: Tue, 3 Jan 2017 10:54:02 +0000 (-0800)
Subject: Revert "[HAM] - fixing pedometer listeners"
X-Git-Tag: submit/tizen_3.0/20170103.113010~1^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f60b0dd58d49a3cb85df7ca63aeda384abf0bcd2;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

Revert "[HAM] - fixing pedometer listeners"

This reverts commit dbe8157f28394a8bba37a3ecab194c67e360209a.

Change-Id: I842ab00deef4102f6af98bc48875c748d2d8be9f
---

diff --git a/src/humanactivitymonitor/humanactivitymonitor_api.js b/src/humanactivitymonitor/humanactivitymonitor_api.js
index 5c36b72b..2d2d65e1 100755
--- a/src/humanactivitymonitor/humanactivitymonitor_api.js
+++ b/src/humanactivitymonitor/humanactivitymonitor_api.js
@@ -354,17 +354,12 @@ HumanActivityMonitorManager.prototype.stop = function(type) {
     {name: 'type', type: types_.ENUM, values: Object.keys(HumanActivityType)}
   ]);
 
+  stopListener('HumanActivityMonitor_'  + args.type,
+               'HumanActivityMonitorManager_stop',
+               { type: args.type });
+
   if (HumanActivityType.PEDOMETER === args.type) {
-    if (pedometerListener && !accumulativePedometerListener) {
-      stopListener('HumanActivityMonitor_PEDOMETER',
-                   'HumanActivityMonitorManager_stop',
-                   { type: HumanActivityType.PEDOMETER });
-    }
     pedometerListener = null;
-  } else {
-    stopListener('HumanActivityMonitor_'  + args.type,
-                 'HumanActivityMonitorManager_stop',
-                 { type: args.type });
   }
 
   if (HumanActivityType.GPS === args.type) {
@@ -387,12 +382,13 @@ HumanActivityMonitorManager.prototype.setAccumulativePedometerListener = functio
 };
 
 HumanActivityMonitorManager.prototype.unsetAccumulativePedometerListener = function() {
-  if (accumulativePedometerListener && !pedometerListener) {
-    stopListener('HumanActivityMonitor_PEDOMETER',
-                 'HumanActivityMonitorManager_stop',
-                 { type: HumanActivityType.PEDOMETER });
-  }
+  var oldPedometerListener = pedometerListener;
+
+  // calling stop() will overwrite pedometerListener, needs to be restored afterwards
+  this.stop(HumanActivityType.PEDOMETER);
+
   accumulativePedometerListener = null;
+  pedometerListener = oldPedometerListener;
 };