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 444293 3ade6d760d37851e0cfcdbc755d98ab21f656983
parent 444292 9181a51ed2e46ef19cb8a888a1bc145a2b3e5d83
child 444294 b5969c9b92fb92951471da6402e73e91c0847c94
push id1618
push userCallek@gmail.com
push dateThu, 11 Jan 2018 17:45:48 +0000
treeherdermozilla-release@882ca853e05a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmystor
bugs1413621
milestone58.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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));
   }