Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7 id:120001 of https:…

Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7 id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in test. It's caused by double deleting the tray bubble view when shutting down test/os without closing the IME menu bubble, which should not happen in real production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.

Committed: https://crrev.com/c6200ab48ee282cae1460dfd1aaeafbf732897d6
Cr-Commit-Position: refs/heads/master@{#414819}

[Azure Wei, 2016-08-26 02:40:13]

Description was changed from

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

BUG=640432
TEST=Verified on local build.
==========

to

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

BUG=640432
TEST=Verified on local build.
==========

[Azure Wei, 2016-08-26 02:40:13]

azurewei@chromium.org changed reviewers:
+ jamescook@chromium.org

[Azure Wei, 2016-08-26 02:42:50]

Description was changed from

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

BUG=640432
TEST=Verified on local build.
==========

to

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

BUG=640432
TEST=Verified on local build.
==========

[Azure Wei, 2016-08-26 02:43:02]

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

[commit-bot: I haz the power, 2016-08-26 02:43:59]

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

[Azure Wei, 2016-08-26 03:08:36]

Description was changed from

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

BUG=640432
TEST=Verified on local build.
==========

to

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.
==========

[Azure Wei, 2016-08-26 03:08:42]

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

[commit-bot: I haz the power, 2016-08-26 03:09:08]

Dry run: 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-08-26 04:04:01]

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

[commit-bot: I haz the power, 2016-08-26 04:04:02]

Dry run: This issue passed the CQ dry run.

[Azure Wei, 2016-08-26 07:10:44]

Please review this cl. Thanks!

[James Cook, 2016-08-26 15:52:41]

LGTM

[Azure Wei, 2016-08-26 21:35:26]

The CQ bit was checked by azurewei@chromium.org

[commit-bot: I haz the power, 2016-08-26 21:36:09]

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-08-26 21:42:53]

Description was changed from

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.
==========

to

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.
==========

[commit-bot: I haz the power, 2016-08-26 21:42:54]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-26 21:46:01]

Description was changed from

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.
==========

to

==========
Revert "Revert of Updates the IME list when the IME has refreshed. (patchset #7
id:120001 of https://codereview.chromium.org/2271483003/ )"

This reverts commit 7017855217bfc5d4ea11d44e526373bd7c571dac.

Reland the cl https://codereview.chromium.org/2271483003/ and fix the crash in
test. It's caused by double deleting the tray bubble view when shutting down
test/os without closing the IME menu bubble, which should not happen in real
production.

Fix the underlying crash problem and add new test:
ImeMenuTrayTest.QuitChromeWithMenuOpen
to cover it.

BUG=640432
TEST=Verified on local build.

Committed: https://crrev.com/c6200ab48ee282cae1460dfd1aaeafbf732897d6
Cr-Commit-Position: refs/heads/master@{#414819}
==========

[commit-bot: I haz the power, 2016-08-26 21:46:02]

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