Revert HttpClientManager, HttpResponseDecoder, fix TPM position

Revert HttpClientManager lock and HttpResponseDecoder leak fix to
match main. Restore m_scheduledUploadTime before LOCKGUARD.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: bmehta001

lib/http/HttpClientManager.cpp

@@ -149,15 +149,8 @@ namespace MAT_NS_BEGIN {
     void HttpClientManager::cancelAllRequests()
     {
         cancelAllRequestsAsync();
-        while (true)
-        {
-            {
-                LOCKGUARD(m_httpCallbacksMtx);
-                if (m_httpCallbacks.empty())
-                    break;
-            }
+        while (!m_httpCallbacks.empty())
             std::this_thread::yield();
-        }
     }
 
     // start async cancellation

lib/http/HttpResponseDecoder.cpp

@@ -67,11 +67,13 @@ namespace MAT_NS_BEGIN {
             break;
 
         case HttpResult_Aborted:
+            ctx->httpResponse = nullptr;
             outcome = Abort;
             break;
 
         case HttpResult_LocalFailure:
         case HttpResult_NetworkFailure:
+            ctx->httpResponse = nullptr;
             outcome = RetryNetwork;
             break;
         }
@@ -127,7 +129,6 @@ namespace MAT_NS_BEGIN {
                 evt.param1 = 0; // response.GetStatusCode();
                 DispatchEvent(evt);
             }
-            delete ctx->httpResponse;
             ctx->httpResponse = nullptr;
             // eventsRejected(ctx); // FIXME: [MG] - investigate why ctx gets corrupt after eventsRejected
             requestAborted(ctx);
@@ -158,8 +159,6 @@ namespace MAT_NS_BEGIN {
                 evt.param1 = response.GetStatusCode();
                 DispatchEvent(evt);
             }
-            delete ctx->httpResponse;
-            ctx->httpResponse = nullptr;
             temporaryNetworkFailure(ctx);
             break;
         }

lib/tpm/TransmissionPolicyManager.cpp

@@ -154,7 +154,7 @@ namespace MAT_NS_BEGIN {
                 // m_isUploadScheduled check does not have to be strictly atomic because
                 // the completion of upload will schedule more uploads as-needed, we only
                 // want to avoid the unnecessary wasteful rescheduling.
-                LOG_TRACE("WAIT  upload %d ms for lat=%d", delta, m_runningLatency);
+                LOG_TRACE("WAIT  upload %d ms for lat=%d", delta, m_runningLatency.load());
                 return;
             }
         }
@@ -173,7 +173,7 @@ namespace MAT_NS_BEGIN {
         {
             m_scheduledUploadTime = PAL::getMonotonicTimeMs() + delay.count();
             m_runningLatency = latency;
-            LOG_TRACE("SCHED upload %d ms for lat=%d", delay.count(), m_runningLatency);
+            LOG_TRACE("SCHED upload %d ms for lat=%d", delay.count(), m_runningLatency.load());
             m_scheduledUpload = PAL::scheduleTask(&m_taskDispatcher, static_cast<unsigned>(delay.count()), this, &TransmissionPolicyManager::uploadAsync, latency);
         }
     }