[ios] Reload the page inside WebTestWithWebState::LoadHtml if necessary.

[ios] Reload the page inside WebTestWithWebState::LoadHtml if necessary.

Reload the page if script execution fails. Page reload was performed
for data URLs and looks like it's also needed for loadHTML:baseURL: API.

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

[Eugene But (OOO till 7-30), 2016-09-13 17:16:47]

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

[commit-bot: I haz the power, 2016-09-13 17:17:01]

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-13 17:24:53]

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

[commit-bot: I haz the power, 2016-09-13 17:58:07]

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

[commit-bot: I haz the power, 2016-09-13 17:58:08]

Dry run: This issue passed the CQ dry run.

[Jackie Quinn, 2016-09-15 17:25:43]

On 2016/09/13 17:24:54, Eugene But wrote:

lgtm, though is there any chance of an infinite recursion here? Just wondering
if there should be some form of timeout...

[Eugene But (OOO till 7-30), 2016-09-15 17:53:46]

On 2016/09/15 17:25:43, Jackie Quinn wrote:
> On 2016/09/13 17:24:54, Eugene But wrote:
> 
> lgtm, though is there any chance of an infinite recursion here? Just wondering
> if there should be some form of timeout...
We did not have infinite recursion problem with data URLs. If it ever happens I
can always change this to iterative approach :)

[Eugene But (OOO till 7-30), 2016-09-15 17:53:52]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-09-15 17:54:21]

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-15 18:49:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-15 18:51:18]

Description was changed from

==========
[ios] Reload the page inside WebTestWithWebState::LoadHtml if necessary.

Reload the page if script execution fails. Page reload was performed
for data URLs and looks like it's also needed for loadHTML:baseURL: API.

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

to

==========
[ios] Reload the page inside WebTestWithWebState::LoadHtml if necessary.

Reload the page if script execution fails. Page reload was performed
for data URLs and looks like it's also needed for loadHTML:baseURL: API.

BUG=643835

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

[commit-bot: I haz the power, 2016-09-15 18:51:19]

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