Destroy DeviceImpl when the underlying UsbDevice is disconnected.

Destroy DeviceImpl when the underlying UsbDevice is disconnected.

A open message pipe to a DeviceImpl represents a physical resource, the
USB device connected to the host. Instead of continuing to support
handling messages sent on this pipe after the device is disconnected it
makes more sense to simply close it.

For efficiency and because DeviceImpls can outlive the DeviceManagerImpl
that created them a new observer interface is added to UsbDevice so that
the DeviceImpl can listed for the removal of the physical device all on
its own.

BUG=492204
Committed: https://crrev.com/1474f931e1e45778c3287bd4f7a20620eafb8ab9
Cr-Commit-Position: refs/heads/master@{#375329}

[Reilly Grant (use Gerrit), 2016-02-12 01:01:42]

reillyg@chromium.org changed reviewers:
+ rockot@chromium.org

[Reilly Grant (use Gerrit), 2016-02-12 01:01:43]

Please take a look.

[Ken Rockot(use gerrit already), 2016-02-12 01:51:52]

Missing files?

[Reilly Grant (use Gerrit), 2016-02-12 02:00:04]

On 2016/02/12 01:51:52, Ken Rockot wrote:
> Missing files?

Ah, I see what happened. My branch was set up with the wrong upstream. The patch
felt a bit small. :P

[Ken Rockot(use gerrit already), 2016-02-12 21:38:55]

lgtm

[Reilly Grant (use Gerrit), 2016-02-12 22:11:25]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-02-12 22:11:25]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/1695643002/#ps40001
(title: "Revert change to web_usb_device_impl.cc.")

[commit-bot: I haz the power, 2016-02-12 22:11:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1695643002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1695643002/40001

[commit-bot: I haz the power, 2016-02-12 22:20:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-12 22:20:49]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Reilly Grant (use Gerrit), 2016-02-13 00:58:26]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-02-13 00:59:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1695643002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1695643002/40001

[commit-bot: I haz the power, 2016-02-13 02:07:52]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-16 22:47:04]

Description was changed from

==========
Destroy DeviceImpl when the underlying UsbDevice is disconnected.

A open message pipe to a DeviceImpl represents a physical resource, the
USB device connected to the host. Instead of continuing to support
handling messages sent on this pipe after the device is disconnected it
makes more sense to simply close it.

For efficiency and because DeviceImpls can outlive the DeviceManagerImpl
that created them a new observer interface is added to UsbDevice so that
the DeviceImpl can listed for the removal of the physical device all on
its own.

BUG=492204
==========

to

==========
Destroy DeviceImpl when the underlying UsbDevice is disconnected.

A open message pipe to a DeviceImpl represents a physical resource, the
USB device connected to the host. Instead of continuing to support
handling messages sent on this pipe after the device is disconnected it
makes more sense to simply close it.

For efficiency and because DeviceImpls can outlive the DeviceManagerImpl
that created them a new observer interface is added to UsbDevice so that
the DeviceImpl can listed for the removal of the physical device all on
its own.

BUG=492204

Committed: https://crrev.com/1474f931e1e45778c3287bd4f7a20620eafb8ab9
Cr-Commit-Position: refs/heads/master@{#375329}
==========

[commit-bot: I haz the power, 2016-02-16 22:47:05]

Patchset 3 (id:??) landed as
https://crrev.com/1474f931e1e45778c3287bd4f7a20620eafb8ab9
Cr-Commit-Position: refs/heads/master@{#375329}