Fix thread safety issues with //components/upload_list.

chrome/browser/crash_upload_list_crashpad.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/crash_upload_list_crashpad.h"
 
 #include <stddef.h>
 
 #include "base/threading/sequenced_worker_pool.h"
 #include "base/time/time.h"
@@ skipping 29 matching lines @@
 
 }  // namespace
 
 CrashUploadListCrashpad::CrashUploadListCrashpad(
     Delegate* delegate,
     const scoped_refptr<base::SequencedWorkerPool>& worker_pool)
     : CrashUploadList(delegate, base::FilePath(), worker_pool) {}
 
 CrashUploadListCrashpad::~CrashUploadListCrashpad() {}
 
-void CrashUploadListCrashpad::LoadUploadList() {
+void CrashUploadListCrashpad::LoadUploadList(
+    std::vector<UploadList::UploadInfo>* uploads) {
   std::vector<crash_reporter::UploadedReport> uploaded_reports;
 #if defined(OS_WIN)
   // On Windows, we only link crash client into chrome.exe (not the dlls), and
   // it does the registration. That means the global that holds the crash report
   // database lives in the .exe, so we need to grab a pointer to a helper in the
   // exe to get our uploaded reports list.
   GetUploadedReportsThunk(&uploaded_reports);
 #else
   crash_reporter::GetUploadedReports(&uploaded_reports);
 #endif
 
-  ClearUploads();
   for (const crash_reporter::UploadedReport& uploaded_report :
        uploaded_reports) {
-    AppendUploadInfo(
+    uploads->push_back(
         UploadInfo(uploaded_report.remote_id,
                    base::Time::FromTimeT(uploaded_report.creation_time),
                    uploaded_report.local_id, base::Time()));
   }
 }