Fix ResumableTCPSocket destruction.

extensions/browser/api/socket/tcp_socket.cc

 // Copyright 2014 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 "extensions/browser/api/socket/tcp_socket.h"
 
 #include "base/callback_helpers.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 340 matching lines @@
 
 ResumableTCPSocket::ResumableTCPSocket(
     std::unique_ptr<net::TCPClientSocket> tcp_client_socket,
     const std::string& owner_extension_id,
     bool is_connected)
     : TCPSocket(std::move(tcp_client_socket), owner_extension_id, is_connected),
       persistent_(false),
       buffer_size_(0),
       paused_(false) {}
 
+ResumableTCPSocket::~ResumableTCPSocket() {
+  // Despite ~TCPSocket doing basically the same, we need to disconnect
+  // before ResumableTCPSocket is destroyed, because we have some extra
+  // state that relies on the socket being ResumableTCPSocket, like
+  // read_callback_.
+  Disconnect();
+}
+
 bool ResumableTCPSocket::IsPersistent() const { return persistent(); }
 
 ResumableTCPServerSocket::ResumableTCPServerSocket(
     const std::string& owner_extension_id)
     : TCPSocket(owner_extension_id), persistent_(false), paused_(false) {}
 
 bool ResumableTCPServerSocket::IsPersistent() const { return persistent(); }
 
 }  // namespace extensions