[Code health] Convert AndroidUsbSocket delete callback to OnceClosure

Bug: 1152274
Change-Id: I179c720aa3b384c243cf61c3186fae17f8acfc59
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2585691Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836798}

chrome/browser/devtools/device/usb/android_usb_device.cc

@@ -233,7 +233,7 @@ net::StreamSocket* AndroidUsbDevice::CreateSocket(const std::string& command) {
   uint32_t socket_id = ++last_socket_id_;
   sockets_[socket_id] = new AndroidUsbSocket(
       this, socket_id, command,
-      base::Bind(&AndroidUsbDevice::SocketDeleted, this, socket_id));
+      base::BindOnce(&AndroidUsbDevice::SocketDeleted, this, socket_id));
   return sockets_[socket_id];
 }
 

chrome/browser/devtools/device/usb/android_usb_socket.cc

@@ -23,20 +23,20 @@ const int kMaxPayload = 4096;
 AndroidUsbSocket::AndroidUsbSocket(scoped_refptr<AndroidUsbDevice> device,
                                    uint32_t socket_id,
                                    const std::string& command,
-                                   base::Closure delete_callback)
+                                   base::OnceClosure delete_callback)
     : device_(device),
       command_(command),
       local_id_(socket_id),
       remote_id_(0),
       is_connected_(false),
-      delete_callback_(delete_callback) {}
+      delete_callback_(std::move(delete_callback)) {}
 
 AndroidUsbSocket::~AndroidUsbSocket() {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   if (is_connected_)
     Disconnect();
   if (!delete_callback_.is_null())
-    delete_callback_.Run();
+    std::move(delete_callback_).Run();
 }
 
 void AndroidUsbSocket::HandleIncoming(std::unique_ptr<AdbMessage> message) {

chrome/browser/devtools/device/usb/android_usb_socket.h

@@ -24,7 +24,7 @@ class AndroidUsbSocket : public net::StreamSocket {
   AndroidUsbSocket(scoped_refptr<AndroidUsbDevice> device,
                    uint32_t socket_id,
                    const std::string& command,
-                   base::Closure delete_callback);
+                   base::OnceClosure delete_callback);
   ~AndroidUsbSocket() override;
 
   void HandleIncoming(std::unique_ptr<AdbMessage> message);
@@ -77,7 +77,7 @@ class AndroidUsbSocket : public net::StreamSocket {
   net::CompletionOnceCallback connect_callback_;
   net::CompletionOnceCallback read_callback_;
   net::CompletionOnceCallback write_callback_;
-  base::Closure delete_callback_;
+  base::OnceClosure delete_callback_;
 
   SEQUENCE_CHECKER(sequence_checker_);