Commit 628bcf8f authored by stkhapugin@chromium.org's avatar stkhapugin@chromium.org Committed by Commit Bot

[iOS] Update the location bar when security state changes.

Update the location string, not only the security icon, when the
security state changes to indicate correct URL on interstitials.

Bug: 853332
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I3ba5e7dcaf26945f394892354127be01fa9564a7
Reviewed-on: https://chromium-review.googlesource.com/1113744
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570070}
parent eaf87951
......@@ -114,6 +114,13 @@
- (void)webStateDidChangeVisibleSecurityState:(web::WebState*)webState {
DCHECK_EQ(_webState, webState);
// Currently, because of https://crbug.com/448486 , interstitials are not
// commited navigations. This means that if a security interstitial (e.g. for
// broken HTTPS) is shown, didFinishNavigation: is not called, and
// didChangeVisibleSecurityState: is the only chance to update the URL.
// Otherwise it would be preferable to only update the icon here.
[self notifyConsumerOfChangedLocation];
[self notifyConsumerOfChangedSecurityIcon];
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment