Bug 1254550 - Make Telemetry configuration consistent across builds. r=gfritzsche, ba=lizzard
authorAlessio Placitelli <alessio.placitelli@gmail.com>
Fri, 11 Mar 2016 06:31:00 +0100
changeset 319543 b19e6561f416b45534f3205c0bc7028dbdac1e7d
parent 319542 bd3a8572be8677053ec95066251bc9efc47b5813
child 319544 a6da2113fc2850238b7d1a7047e89841b8a52ae1
push id1079
push userjlund@mozilla.com
push dateFri, 15 Apr 2016 21:02:33 +0000
treeherdermozilla-release@575fbf6786d5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgfritzsche
bugs1254550
milestone46.0
Bug 1254550 - Make Telemetry configuration consistent across builds. r=gfritzsche, ba=lizzard
toolkit/components/telemetry/Telemetry.cpp
toolkit/components/telemetry/nsITelemetry.idl
--- a/toolkit/components/telemetry/Telemetry.cpp
+++ b/toolkit/components/telemetry/Telemetry.cpp
@@ -3339,17 +3339,17 @@ TelemetryImpl::SetCanRecordExtended(bool
  */
 bool
 TelemetryImpl::CanRecordExtended() {
   return !sTelemetry || sTelemetry->mCanRecordExtended;
 }
 
 NS_IMETHODIMP
 TelemetryImpl::GetIsOfficialTelemetry(bool *ret) {
-#if defined(MOZILLA_OFFICIAL) && defined(MOZ_TELEMETRY_REPORTING)
+#if defined(MOZILLA_OFFICIAL) && defined(MOZ_TELEMETRY_REPORTING) && !defined(DEBUG)
   *ret = true;
 #else
   *ret = false;
 #endif
   return NS_OK;
 }
 
 already_AddRefed<nsITelemetry>
--- a/toolkit/components/telemetry/nsITelemetry.idl
+++ b/toolkit/components/telemetry/nsITelemetry.idl
@@ -275,18 +275,18 @@ interface nsITelemetry : nsISupports
    * set to false during tests.
    *
    * Set this to false in tests to disable gathering of extended telemetry statistics.
    */
   attribute boolean canRecordExtended;
 
   /**
    * A flag indicating whether Telemetry can submit official results (for base or extended
-   * data). This is true on official builds with built in support for Mozilla Telemetry
-   * reporting.
+   * data). This is true on official, non-debug builds with built in support for Mozilla
+   * Telemetry reporting.
    */
   readonly attribute boolean isOfficialTelemetry;
 
   /** Addon telemetry hooks */
 
   /**
    * Register a histogram for an addon.  Throws an error if the
    * histogram name has been registered previously.