[ios] Fixed WebTestWithWebState::LoadHtml flakyness.

[ios] Fixed WebTestWithWebState::LoadHtml flakyness.

Polling for PAGE_LOADING loadPhase is flaky because the phase can
quickly change to PAGE_LOADED in the same runloop cycle. Use
WebStateObserver to wait until navigation is committed instead.

BUG=643835
Committed: https://crrev.com/1aae82276243c4daa1f0604dab1adcf279112229
Cr-Commit-Position: refs/heads/master@{#416953}

[Eugene But (OOO till 7-30), 2016-09-07 04:11:13]

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

[commit-bot: I haz the power, 2016-09-07 04:11:26]

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

[Eugene But (OOO till 7-30), 2016-09-07 04:45:15]

eugenebut@chromium.org changed reviewers:
+ marq@chromium.org

[Eugene But (OOO till 7-30), 2016-09-07 04:45:16]

This should make our tree happier

[commit-bot: I haz the power, 2016-09-07 05:04:25]

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

[commit-bot: I haz the power, 2016-09-07 05:04:26]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-09-07 15:56:37]

eugenebut@chromium.org changed reviewers:
+ jyquinn@chromium.org

[Jackie Quinn, 2016-09-07 16:15:05]

lgtm!

[Eugene But (OOO till 7-30), 2016-09-07 16:16:33]

Thanks for a quick review!

[Eugene But (OOO till 7-30), 2016-09-07 16:16:38]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-09-07 16:17:05]

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-09-07 16:21:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-07 16:23:49]

Description was changed from

==========
[ios] Fixed WebTestWithWebState::LoadHtml flakyness.

Polling for PAGE_LOADING loadPhase is flaky because the phase can
quickly change to PAGE_LOADED in the same runloop cycle. Use
WebStateObserver to wait until navigation is committed instead.

BUG=643835
==========

to

==========
[ios] Fixed WebTestWithWebState::LoadHtml flakyness.

Polling for PAGE_LOADING loadPhase is flaky because the phase can
quickly change to PAGE_LOADED in the same runloop cycle. Use
WebStateObserver to wait until navigation is committed instead.

BUG=643835

Committed: https://crrev.com/1aae82276243c4daa1f0604dab1adcf279112229
Cr-Commit-Position: refs/heads/master@{#416953}
==========

[commit-bot: I haz the power, 2016-09-07 16:23:50]

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