[ios] Restore the original key window at the end of the AlertCoordinator tests.

[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None

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

[rohitrao (ping after 24h), 2016-12-20 02:06:41]

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

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

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

[sdefresne, 2016-12-20 02:08:55]

sdefresne@chromium.org changed reviewers:
+ sdefresne@chromium.org

[sdefresne, 2016-12-20 02:08:55]

https://codereview.chromium.org/2588223002/diff/1/ios/chrome/browser/ui/alert...
File ios/chrome/browser/ui/alert_coordinator/alert_coordinator_unittest.mm
(right):

https://codereview.chromium.org/2588223002/diff/1/ios/chrome/browser/ui/alert...
ios/chrome/browser/ui/alert_coordinator/alert_coordinator_unittest.mm:34: void
TearDown() override { [previous_key_window_ makeKeyAndVisible]; }
Should be in the destructor since the recording is in the constructor.

[lpromero, 2016-12-20 02:09:43]

lpromero@chromium.org changed reviewers:
+ lpromero@chromium.org

[lpromero, 2016-12-20 02:09:44]

lgtm

[commit-bot: I haz the power, 2016-12-20 02:16:59]

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

[commit-bot: I haz the power, 2016-12-20 02:17:00]

Dry run: This issue passed the CQ dry run.

[rohitrao (ping after 24h), 2016-12-20 02:31:01]

Description was changed from

==========
[ios] Restore the original key window at the end of the AlertCoordinator tests.

Leaving the app without a key window breaks subsequent tests on iOS 9.0,
including ContextualSearchJsTest.

BUG=None
==========

to

==========
[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None
==========

[rohitrao (ping after 24h), 2016-12-20 02:31:50]

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

[commit-bot: I haz the power, 2016-12-20 02:32:16]

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-12-20 02:41:22]

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

[commit-bot: I haz the power, 2016-12-20 02:41:23]

Dry run: This issue passed the CQ dry run.

[rohitrao (ping after 24h), 2016-12-20 02:42:04]

The CQ bit was checked by rohitrao@chromium.org

[rohitrao (ping after 24h), 2016-12-20 02:42:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
lpromero@chromium.org
Link to the patchset: https://codereview.chromium.org/2588223002/#ps20001
(title: "One more test.")

[commit-bot: I haz the power, 2016-12-20 02:42:32]

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-20 02:46:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482201724770110,
"parent_rev": "87853db33b4cde9bea0d9feb593e7c2cc2a3035f", "commit_rev":
"805341cf3c0f8590df71cbb5f35325d0b6444740"}

[commit-bot: I haz the power, 2016-12-20 02:46:40]

Description was changed from

==========
[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None
==========

to

==========
[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None

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

[commit-bot: I haz the power, 2016-12-20 02:46:41]

Committed patchset #2 (id:20001)

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

Description was changed from

==========
[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None

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

to

==========
[ios] Restore the original key window at the end of unittests.

Some tests were setting a new key window but failing to restore the original
window after the test concluded.  Leaving the app without a key window breaks
subsequent tests on iOS 9.0, including ContextualSearchJsTest.

BUG=None

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

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

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