Remove ArcImeService from window observers on shutdown

Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
TEST=none

Review-Url: https://codereview.chromium.org/2709113002
Cr-Commit-Position: refs/heads/master@{#452417}
Committed: https://chromium.googlesource.com/chromium/src/+/a6e727ae6c9a9a9994a87ebb2e1025e9982e15c7

[yoshiki, 2017-02-22 06:51:05]

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

[yoshiki, 2017-02-22 06:52:07]

Description was changed from

==========
wip

.

BUG=691842
==========

to

==========
Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
==========

[yoshiki, 2017-02-22 06:52:07]

yoshiki@chromium.org changed reviewers:
+ kinaba@chromium.org

[yoshiki, 2017-02-22 06:52:14]

Description was changed from

==========
Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
==========

to

==========
Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
TEST=none
==========

[commit-bot: I haz the power, 2017-02-22 06:52:15]

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

[yoshiki, 2017-02-22 06:53:52]

kinaba@, PTAL.

[kinaba, 2017-02-22 07:02:07]

thanks! lgtm but you need OWNER review too

[yoshiki, 2017-02-22 07:20:29]

yoshiki@chromium.org changed reviewers:
+ hidehiko@chromium.org

[yoshiki, 2017-02-22 07:20:30]

Hidehiko, could you approve this?

[commit-bot: I haz the power, 2017-02-22 08:16:11]

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

[commit-bot: I haz the power, 2017-02-22 08:16:12]

Dry run: This issue passed the CQ dry run.

[hidehiko, 2017-02-23 05:18:03]

lgtm

[yoshiki, 2017-02-23 06:11:48]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2017-02-23 06:12:12]

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, 2017-02-23 06:16:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487830308262180, "parent_rev":
"a5284b9c11fb616ff61bc3343584409cf1e558fa", "commit_rev":
"a6e727ae6c9a9a9994a87ebb2e1025e9982e15c7"}

[commit-bot: I haz the power, 2017-02-23 06:17:56]

Description was changed from

==========
Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
TEST=none
==========

to

==========
Remove ArcImeService from window observers on shutdown

Remaining observer fails CHECK and causes crash. This patch prevents it.

BUG=691842
TEST=none

Review-Url: https://codereview.chromium.org/2709113002
Cr-Commit-Position: refs/heads/master@{#452417}
Committed:
https://chromium.googlesource.com/chromium/src/+/a6e727ae6c9a9a9994a87ebb2e10...
==========

[commit-bot: I haz the power, 2017-02-23 06:17:57]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a6e727ae6c9a9a9994a87ebb2e10...