[ios] Fixed SSL Interstitial reload.

[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly

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

[Eugene But (OOO till 7-30), 2016-10-26 21:12:03]

Description was changed from

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
==========

to

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly
==========

[Eugene But (OOO till 7-30), 2016-10-26 21:12:46]

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

[commit-bot: I haz the power, 2016-10-26 21:14: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-10-26 21:25:24]

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

[commit-bot: I haz the power, 2016-10-26 21:25:25]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-10-26 21:37:44]

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

[kkhorimoto, 2016-10-27 00:15:18]

The CQ bit was checked by kkhorimoto@chromium.org

[kkhorimoto, 2016-10-27 00:15:18]

lgtm

[commit-bot: I haz the power, 2016-10-27 00:15:49]

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-10-27 00:21:11]

Description was changed from

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly
==========

to

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly
==========

[commit-bot: I haz the power, 2016-10-27 00:21:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-27 00:23:39]

Description was changed from

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly
==========

to

==========
[ios] Fixed SSL Interstitial reload.

Do not reload inside IOSSSLErrorHandler::InterstitialWasDismissed to
avoid extra reloads.

InterstitialWasDismissed is called in the following cases:
1.) Reload button tapped (no need for the second reload)
2.) Back button tapped (no need to reload, going back is enough)
3.) Tab closed (no need to reload closed tab)
4.) "Back to safety" button tapped (reload is actually needed in this
    case, but it can be done in WebInterstitialImpl::DontProceed).

BUG=653963
TEST=dismissing SSL interstitial works correctly

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

[commit-bot: I haz the power, 2016-10-27 00:23:40]

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