Bug 1413621 - Telemetry: replace NS_WARN_IF with MOZ_ASSERT. r=mystor
authorPaul Bignier <paul.bignier@gmail.com>
Thu, 09 Nov 2017 00:33:00 -0500
changeset 695876 3ade6d760d37851e0cfcdbc755d98ab21f656983
parent 695875 9181a51ed2e46ef19cb8a888a1bc145a2b3e5d83
child 695877 b5969c9b92fb92951471da6402e73e91c0847c94
push id88577
push usermozilla@noorenberghe.ca
push dateThu, 09 Nov 2017 23:43:51 +0000
reviewersmystor
bugs1413621
milestone58.0a1
Bug 1413621 - Telemetry: replace NS_WARN_IF with MOZ_ASSERT. r=mystor
toolkit/components/telemetry/ipc/TelemetryIPCAccumulator.cpp
--- a/toolkit/components/telemetry/ipc/TelemetryIPCAccumulator.cpp
+++ b/toolkit/components/telemetry/ipc/TelemetryIPCAccumulator.cpp
@@ -285,17 +285,17 @@ SendAccumulatedData(TActor* ipcActor)
     if (gChildEvents) {
       eventsToSend.SwapElements(*gChildEvents);
     }
     discardedData = gDiscardedData;
     gDiscardedData = {0};
   }
 
   // Send the accumulated data to the parent process.
-  mozilla::Unused << NS_WARN_IF(!ipcActor);
+  MOZ_ASSERT(ipcActor);
   if (histogramsToSend.Length()) {
     mozilla::Unused <<
       NS_WARN_IF(!ipcActor->SendAccumulateChildHistograms(histogramsToSend));
   }
   if (keyedHistogramsToSend.Length()) {
     mozilla::Unused <<
       NS_WARN_IF(!ipcActor->SendAccumulateChildKeyedHistograms(keyedHistogramsToSend));
   }