Manual crash uploads for mac and win

components/upload_list/upload_list.cc

Index: components/upload_list/upload_list.cc
diff --git a/components/upload_list/upload_list.cc b/components/upload_list/upload_list.cc
index cdaf6b16678af9655d65e9091e1879b783dbfc66..c4768ea6d9d8c1b1d0fee49712e867e594288fce 100644
--- a/components/upload_list/upload_list.cc
+++ b/components/upload_list/upload_list.cc
@@ -133,3 +133,18 @@ void UploadList::GetUploads(size_t max_count,
             uploads_.begin() + std::min(uploads_.size(), max_count),
             std::back_inserter(*uploads));
 }
+
+void UploadList::RequestSingleCrashUploadAsync(const std::string& local_id) {
+#if defined(OS_WIN) || defined(OS_MACOSX)
+  DCHECK(thread_checker_.CalledOnValidThread());
+  worker_pool_->PostTask(
+      FROM_HERE,
+      base::Bind(&UploadList::RequestSingleCrashUpload, this, local_id));
+#endif
+}
+
+void UploadList::RequestSingleCrashUpload(const std::string& local_id) {
+  // Manual uploads for not uploaded crash reports are not availble for non

[jwd, 2016/08/23 19:14:30]

availble -> available

[gayane -on leave until 09-2017, 2016/08/23 20:57:45]

Done.

+  // crashpad systems.
+  NOTREACHED();

[jwd, 2016/08/23 19:14:30]

I wonder if this function should be defined here at all. You could make
RequestSingleCrashUploadAsync virtual with the NOTREACHED, and just implement it
properly in the crashpad version, and have a RequestSingleCrashUpload defined
there only, no?

[gayane -on leave until 09-2017, 2016/08/23 20:57:45]

initially i did it like you suggested, but then I thought that if we are going
to add Android part then we will just need to implementRequestSingleCrashUpload
in here and Async will be common for all the platforms.

[jwd, 2016/08/24 15:09:19]

Ok, can you add a comment that this is here to support android later?

+}