11622 : OutlierDetection should use Ticker, not TimeProvider

Author: vimanikag

netty/src/main/java/io/grpc/netty/AbstractNettyHandler.java

@@ -199,8 +199,7 @@ public void updateWindow() throws Http2Exception {
       pingReturn++;
       setPinging(false);
 
-      long currentTickerTimeNanos = ticker.read();
-      long elapsedTime = Math.max(0L,(currentTickerTimeNanos - lastPingTime));
+      long elapsedTime = (ticker.read() - lastPingTime);
       if (elapsedTime == 0) {
         elapsedTime = 1;
       }

util/src/main/java/io/grpc/util/OutlierDetectionLoadBalancer.java

@@ -148,12 +148,10 @@ public Status acceptResolvedAddresses(ResolvedAddresses resolvedAddresses) {
         // On the first go we use the configured interval.
         initialDelayNanos = config.intervalNanos;
       } else {
-        long currentTickerTimeNanos = ticker.read();
-        long elapsedTimeNanos = Math.max(0L,(currentTickerTimeNanos - detectionTimerStartNanos));
         // If a timer has started earlier we cancel it and use the difference between the start
         // time and now as the interval.
         initialDelayNanos = Math.max(0L,
-            config.intervalNanos - elapsedTimeNanos);
+                config.intervalNanos - Math.max(0L,(ticker.read() - detectionTimerStartNanos)));
       }
 
       // If a timer has been previously created we need to cancel it and reset all the call counters