Use new SimpleRequest for each DM error report.

Author: sorinj

omaha/goopdate/dm_client.cc

@@ -295,8 +295,9 @@ HRESULT FetchPolicies(HttpRequestInterface* http_request,
   hr = ParseDevicePolicyResponse(response, info, dm_token, device_id, responses,
                                  &validation_results);
   for (const PolicyValidationResult& validation_result : validation_results) {
-    SendPolicyValidationResultReportIfNeeded(http_request, dm_token, device_id,
-                                             validation_result);
+    // SendPolicyValidationResultReportIfNeeded owns the created SimpleRequest.
+    SendPolicyValidationResultReportIfNeeded(new SimpleRequest, dm_token,
+                                             device_id, validation_result);
   }
 
   if (FAILED(hr)) {

omaha/goopdate/dm_messages.cc

@@ -337,7 +337,8 @@ bool ValidatePolicyFetchResponse(
 enterprise_management::PolicyValidationReportRequest::ValidationResultType
 TranslatePolicyValidationResult(PolicyValidationResult::Status status) {
   using Report = enterprise_management::PolicyValidationReportRequest;
-  const std::map<PolicyValidationResult::Status, Report::ValidationResultType>
+  const static std::map<PolicyValidationResult::Status,
+                        Report::ValidationResultType>
       kValidationStatusMap = {
           {PolicyValidationResult::Status::kValidationOK,
            Report::VALIDATION_RESULT_TYPE_SUCCESS},