Do not delete blink::WebRTCCertificateCallback on non-main thread

base/sequenced_task_runner.h

 // Copyright (c) 2012 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.
 
 #ifndef BASE_SEQUENCED_TASK_RUNNER_H_
 #define BASE_SEQUENCED_TASK_RUNNER_H_
 
 #include "base/base_export.h"
 #include "base/sequenced_task_runner_helpers.h"
 #include "base/task_runner.h"
@@ skipping 136 matching lines @@
 
   bool DeleteSoonInternal(const tracked_objects::Location& from_here,
                           void(*deleter)(const void*),
                           const void* object);
 
   bool ReleaseSoonInternal(const tracked_objects::Location& from_here,
                            void(*releaser)(const void*),
                            const void* object);
 };
 
+struct BASE_EXPORT OnTaskRunnerDeleter {
+  explicit OnTaskRunnerDeleter(scoped_refptr<SequencedTaskRunner> task_runner);
+  ~OnTaskRunnerDeleter();
+
+  OnTaskRunnerDeleter(OnTaskRunnerDeleter&&);
+  OnTaskRunnerDeleter& operator=(OnTaskRunnerDeleter&&);
+
+  template <typename T>
+  void operator()(const T* ptr) {
+    if (task_runner_->RunsTasksOnCurrentThread())
+      delete ptr;
+    else if (ptr)
+      task_runner_->DeleteSoon(FROM_HERE, ptr);
+  }
+
+  scoped_refptr<SequencedTaskRunner> task_runner_;
+};

[dcheng, 2016/07/22 07:29:00]

Do you anticipate this being used in a bunch of other places?

[tzik, 2016/07/22 08:14:53]

Yes. I saw several places that hold an object on a foreign thread as a
unique_ptr and post it back to the original thread for deletion.
They tend to ignore PostTask failure, and that causes object deletion on the
foreign thread.
I think we should use this deleter for them.

+
 }  // namespace base
 
 #endif  // BASE_SEQUENCED_TASK_RUNNER_H_