[ios] Set WebState::IsLoading flag to false if load is cancelled.

[ios] Set WebState::IsLoading flag to false if load is cancelled.

Before this change WebState::IsLoading was returning false which caused
Toolbar to think that page did not finish loading. In M52 and earlier
toolbar was relying to CRWWebController.loadPhase API to check the load
state so this bug is M53 regression.

Keeping WebState::IsLoading in sync with CRWWebController.loadPhase
fixes toobar bug and is generally a reasonable thing to do.

BUG=638277
Committed: https://crrev.com/16e4de4de55c5c946b8bdfa47bfee274afc930f2
Cr-Commit-Position: refs/heads/master@{#412391}

[Eugene But (OOO till 7-30), 2016-08-16 23:01:10]

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

[commit-bot: I haz the power, 2016-08-16 23:02:29]

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-08-16 23:06:48]

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

[kkhorimoto, 2016-08-16 23:09:53]

lgtm

[commit-bot: I haz the power, 2016-08-16 23:57:54]

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

[commit-bot: I haz the power, 2016-08-16 23:57:56]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-08-17 00:03:52]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-08-17 00:04:40]

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-17 00:09:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-17 00:11:37]

Description was changed from

==========
[ios] Set WebState::IsLoading flag to false if load is cancelled.

Before this change WebState::IsLoading was returning false which caused
Toolbar to think that page did not finish loading. In M52 and earlier
toolbar was relying to CRWWebController.loadPhase API to check the load
state so this bug is M53 regression.

Keeping WebState::IsLoading in sync with CRWWebController.loadPhase
fixes toobar bug and is generally a reasonable thing to do.

BUG=638277
==========

to

==========
[ios] Set WebState::IsLoading flag to false if load is cancelled.

Before this change WebState::IsLoading was returning false which caused
Toolbar to think that page did not finish loading. In M52 and earlier
toolbar was relying to CRWWebController.loadPhase API to check the load
state so this bug is M53 regression.

Keeping WebState::IsLoading in sync with CRWWebController.loadPhase
fixes toobar bug and is generally a reasonable thing to do.

BUG=638277

Committed: https://crrev.com/16e4de4de55c5c946b8bdfa47bfee274afc930f2
Cr-Commit-Position: refs/heads/master@{#412391}
==========

[commit-bot: I haz the power, 2016-08-17 00:11:38]

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