Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1744)

Unified Diff: ios/chrome/browser/tabs/tab.mm

Issue 2752013002: Removed webControllerDidUpdateSSLStatusForCurrentNavigationItem:. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | ios/web/public/test/fakes/crw_test_web_state_observer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/tabs/tab.mm
diff --git a/ios/chrome/browser/tabs/tab.mm b/ios/chrome/browser/tabs/tab.mm
index 96c7d6b1f8e0708878a068430e2f536ea3768da3..65dc2eb6be6a4ba1207763d62ca02d1dfba6597f 100644
--- a/ios/chrome/browser/tabs/tab.mm
+++ b/ios/chrome/browser/tabs/tab.mm
@@ -1913,14 +1913,14 @@ void TabInfoBarObserver::OnInfoBarReplaced(infobars::InfoBar* old_infobar,
return [self.tabHeadersDelegate headerHeightForTab:self];
}
-- (void)webControllerDidUpdateSSLStatusForCurrentNavigationItem:
- (CRWWebController*)webController {
+- (void)webStateDidChangeVisibleSecurityState:(web::WebState*)webState {
// Disable fullscreen if SSL cert is invalid.
web::NavigationItem* item = [self navigationManager]->GetTransientItem();
- web::SecurityStyle securityStyle =
- item ? item->GetSSL().security_style : web::SECURITY_STYLE_UNKNOWN;
- if (securityStyle == web::SECURITY_STYLE_AUTHENTICATION_BROKEN) {
- [fullScreenController_ disableFullScreen];
+ if (item) {
+ web::SecurityStyle securityStyle = item->GetSSL().security_style;
+ if (securityStyle == web::SECURITY_STYLE_AUTHENTICATION_BROKEN) {
+ [fullScreenController_ disableFullScreen];
+ }
}
[parentTabModel_ notifyTabChanged:self];
« no previous file with comments | « no previous file | ios/web/public/test/fakes/crw_test_web_state_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698