Clean up: no need to change 0 to 1 as data pipeline handles 0 now. (#…

…9593)
ampproject · May 30, 2017 · 0dfd8ef · 0dfd8ef
1 parent 05b8472
commit 0dfd8ef
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/service/performance-impl.js b/src/service/performance-impl.js
@@ -286,8 +286,8 @@ export class Performance {
     const data = {
       label,
       value,
-      // Delta can be 0 or negative, but will always be changed to 1.
-      delta: opt_delta != null ? Math.max(opt_delta, 1) : undefined,
+      // Delta can negative, but will always be changed to 0.
+      delta: opt_delta != null ? Math.max(opt_delta, 0) : undefined,
     };
     if (this.isMessagingReady_ && this.isPerformanceTrackingOn_) {
       this.viewer_.sendMessage('tick', data);

diff --git a/test/functional/test-performance.js b/test/functional/test-performance.js
@@ -73,7 +73,7 @@ describes.realWin('performance', {amp: true}, env => {
       expect(perf.events_[c])
           .to.be.jsonEqual({
             label: 'test1',
-            delta: 1,
+            delta: 0,
           });
 
       c++;
@@ -82,7 +82,7 @@ describes.realWin('performance', {amp: true}, env => {
       expect(perf.events_[c])
           .to.be.jsonEqual({
             label: 'test2',
-            delta: 1,
+            delta: 0,
           });
 
       c++;