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

base/sequenced_task_runner.cc

 // 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.
 
 #include "base/sequenced_task_runner.h"
 
 #include "base/bind.h"
 
 namespace base {
 
@@ skipping 10 matching lines @@
   return PostNonNestableTask(from_here, Bind(deleter, object));
 }
 
 bool SequencedTaskRunner::ReleaseSoonInternal(
     const tracked_objects::Location& from_here,
     void(*releaser)(const void*),
     const void* object) {
   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