Bug 1366147 - SetTarget() should be called once only after the timer is created. r=billm

MozReview-Commit-ID: 9kmxzZX8Zfc

--HG--
extra : rebase_source : d486637b7fe74dbb0544b63abf89bf3f9c4c8d3d

toolkit/components/telemetry/ipc/TelemetryIPCAccumulator.cpp

@@ -81,9 +81,11 @@ DoArmIPCTimerMainThread(const StaticMutexAutoLock& lock)
   }
   if (!gIPCTimer) {
     CallCreateInstance(NS_TIMER_CONTRACTID, &gIPCTimer);
+    if (gIPCTimer) {
+      gIPCTimer->SetTarget(SystemGroup::EventTargetFor(TaskCategory::Other));
+    }
   }
   if (gIPCTimer) {
-    gIPCTimer->SetTarget(SystemGroup::EventTargetFor(TaskCategory::Other));
     gIPCTimer->InitWithNamedFuncCallback(TelemetryIPCAccumulator::IPCTimerFired,
                                          nullptr, kBatchTimeoutMs,
                                          nsITimer::TYPE_ONE_SHOT,