[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

InnerDetach destroyes the current session and domain handlers, as opposite to
NotifyDetached, which prevents stale pointers and inconsistent state at the time
of DTAHI destruction.

BUG=674474
Committed: https://crrev.com/6ac97c212042cc7f4f3b7c3033c8ef4392a133ed
Cr-Commit-Position: refs/heads/master@{#439003}

[dgozman, 2016-12-15 23:24:14]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2016-12-15 23:24:15]

Could you please take a look?

[dgozman, 2016-12-15 23:24:19]

The CQ bit was checked by dgozman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-15 23:25:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pfeldman, 2016-12-16 00:19:57]

lgtm

[commit-bot: I haz the power, 2016-12-16 01:11:49]

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

[commit-bot: I haz the power, 2016-12-16 01:11:50]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-12-16 02:08:03]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-12-16 02:09:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-16 03:36:33]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481854083848150,
"parent_rev": "a6f96c9c1c1234246564c9687daa159d125237e5", "commit_rev":
"3508d244e1728f26ad7bb9171ab49f76c0949642"}

[commit-bot: I haz the power, 2016-12-16 03:36:58]

Description was changed from

==========
[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

InnerDetach destroyes the current session and domain handlers, as opposite to
NotifyDetached, which prevents stale pointers and inconsistent state at the time
of DTAHI destruction.

BUG=674474
==========

to

==========
[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

InnerDetach destroyes the current session and domain handlers, as opposite to
NotifyDetached, which prevents stale pointers and inconsistent state at the time
of DTAHI destruction.

BUG=674474

Review-Url: https://codereview.chromium.org/2579923002
==========

[commit-bot: I haz the power, 2016-12-16 03:36:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-16 03:40:39]

Description was changed from

==========
[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

InnerDetach destroyes the current session and domain handlers, as opposite to
NotifyDetached, which prevents stale pointers and inconsistent state at the time
of DTAHI destruction.

BUG=674474

Review-Url: https://codereview.chromium.org/2579923002
==========

to

==========
[DevTools] Call InnerDetach instead of NotifyDetached in DTAHI::HostClosed.

InnerDetach destroyes the current session and domain handlers, as opposite to
NotifyDetached, which prevents stale pointers and inconsistent state at the time
of DTAHI destruction.

BUG=674474

Committed: https://crrev.com/6ac97c212042cc7f4f3b7c3033c8ef4392a133ed
Cr-Commit-Position: refs/heads/master@{#439003}
==========

[commit-bot: I haz the power, 2016-12-16 03:40:41]

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