Bug 1142079 - Disable refresh driver telemetry on Android. r=froydnj, a=lmandel
authorAvi Halachmi <avihpit@yahoo.com>
Wed, 18 Mar 2015 08:59:01 +0200
changeset 250448 17adc07baf56
parent 250447 45897d27ef82
child 250449 c1356c27fa1b
push id4591
push userryanvm@gmail.com
push date2015-03-19 18:13 +0000
treeherdermozilla-beta@9b7aa96d0e11 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj, lmandel
bugs1142079
milestone37.0
Bug 1142079 - Disable refresh driver telemetry on Android. r=froydnj, a=lmandel
layout/base/nsRefreshDriver.cpp
--- a/layout/base/nsRefreshDriver.cpp
+++ b/layout/base/nsRefreshDriver.cpp
@@ -294,17 +294,19 @@ protected:
     }
 #endif
 
     // log info & lateness
     LOG("[%p] precise timer last tick late by %f ms, next tick in %d ms",
         this,
         (aNowTime - mTargetTime).ToMilliseconds(),
         delay);
+#ifndef ANDROID  /* bug 1142079 */
     Telemetry::Accumulate(Telemetry::FX_REFRESH_DRIVER_FRAME_DELAY_MS, (aNowTime - mTargetTime).ToMilliseconds());
+#endif
 
     // then schedule the timer
     LOG("[%p] scheduling callback for %d ms (2)", this, delay);
     mTimer->InitWithFuncCallback(TimerTick, this, delay, nsITimer::TYPE_ONE_SHOT);
 
     mTargetTime = newTarget;
   }
 };
@@ -409,17 +411,19 @@ protected:
     // Make sure the delay is never negative.
     uint32_t delay = static_cast<uint32_t>(delayMs < 0 ? 0 : delayMs);
 
     // log info & lateness
     LOG("[%p] precise dwm-vsync timer last tick late by %f ms, next tick in %d ms",
         this,
         (aNowTime - mTargetTime).ToMilliseconds(),
         delay);
+#ifndef ANDROID  /* bug 1142079 */
     Telemetry::Accumulate(Telemetry::FX_REFRESH_DRIVER_FRAME_DELAY_MS, (aNowTime - mTargetTime).ToMilliseconds());
+#endif
 
     // then schedule the timer
     LOG("[%p] scheduling callback for %d ms (2)", this, delay);
     mTimer->InitWithFuncCallback(TimerTick, this, delay, nsITimer::TYPE_ONE_SHOT);
 
     mTargetTime = newTarget;
   }
 };
@@ -1400,17 +1404,19 @@ nsRefreshDriver::Tick(int64_t aNowEpoch,
     profiler_tracing("Paint", "DisplayList", TRACING_INTERVAL_END);
 
     if (nsContentUtils::XPConnect()) {
       nsContentUtils::XPConnect()->NotifyDidPaint();
       nsJSContext::NotifyDidPaint();
     }
   }
 
+#ifndef ANDROID  /* bug 1142079 */
   mozilla::Telemetry::AccumulateTimeDelta(mozilla::Telemetry::REFRESH_DRIVER_TICK, mTickStart);
+#endif
 
   for (uint32_t i = 0; i < mPostRefreshObservers.Length(); ++i) {
     mPostRefreshObservers[i]->DidRefresh();
   }
 
   NS_ASSERTION(mInRefresh, "Still in refresh");
 }