Mojo: Make device/ to not rely on connecton error notification on message loop destruction.

device/usb/mojo/device_impl.cc

 // Copyright 2015 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 "device/usb/mojo/device_impl.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 120 matching lines @@
                        mojo::InterfaceRequest<Device> request)
     : device_(device),
       device_info_(std::move(device_info)),
       permission_provider_(permission_provider),
       observer_(this),
       binding_(this, std::move(request)),
       weak_factory_(this) {
   DCHECK(device_);
   // This object owns itself and will be destroyed if,
   //  * the device is disconnected or
-  //  * the message pipe it is bound to is closed
+  //  * the message pipe it is bound to is closed or the message loop is
+  //  * destructed.
   observer_.Add(device_.get());
-  binding_.set_connection_error_handler([this]() { delete this; });
 }
 
 DeviceImpl::~DeviceImpl() {
   CloseHandle();
 }
 
 void DeviceImpl::CloseHandle() {
   if (device_handle_) {
     device_handle_->Close();
     if (permission_provider_)
@@ skipping 296 matching lines @@
       base::Bind(&OnIsochronousTransferOut, base::Passed(&callback_ptr)));
 }
 
 void DeviceImpl::OnDeviceRemoved(scoped_refptr<UsbDevice> device) {
   DCHECK_EQ(device_, device);
   delete this;
 }
 
 }  // namespace usb
 }  // namespace device