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

base/sequenced_task_runner.cc

Index: base/sequenced_task_runner.cc
diff --git a/base/sequenced_task_runner.cc b/base/sequenced_task_runner.cc
index 00d404881597a1d1c65efd0939523f80e15e68b4..f073ccc18d865cece3529ae447d41a1a098a5731 100644
--- a/base/sequenced_task_runner.cc
+++ b/base/sequenced_task_runner.cc
@@ -28,4 +28,17 @@ bool SequencedTaskRunner::ReleaseSoonInternal(
   return PostNonNestableTask(from_here, Bind(releaser, object));
 }
 
+OnTaskRunnerDeleter::OnTaskRunnerDeleter(
+    scoped_refptr<SequencedTaskRunner> task_runner)
+    : task_runner_(task_runner) {

[dcheng, 2016/07/22 06:12:04]

Nit: std::move()

[tzik, 2016/07/22 07:27:21]

Done.

+}
+
+OnTaskRunnerDeleter::~OnTaskRunnerDeleter() {
+}
+
+OnTaskRunnerDeleter::OnTaskRunnerDeleter(OnTaskRunnerDeleter&&) = default;
+
+OnTaskRunnerDeleter& OnTaskRunnerDeleter::operator=(
+    OnTaskRunnerDeleter&&) = default;

[hbos_chromium, 2016/07/20 12:42:32]

Why do you need to explicitly define these if they are default-implementations?

[tzik, 2016/07/20 14:37:50]

This was to (implicitly) make this class move-only. However it seems harmless to
make this class copyable. Removed these move ctor and assign op.

+
 }  // namespace base