Componentize SecurityStateModel

chrome/browser/ssl/security_state_model_android.cc

Index: chrome/browser/ssl/security_state_model_android.cc
diff --git a/chrome/browser/ssl/security_state_model_android.cc b/chrome/browser/ssl/security_state_model_android.cc
index 34ea06e876df196e0bef09c23d157cc7dbc0dbfa..a4c48e1a6c81d352933b18a1576c2f7438bf9868 100644
--- a/chrome/browser/ssl/security_state_model_android.cc
+++ b/chrome/browser/ssl/security_state_model_android.cc
@@ -5,7 +5,7 @@
 #include "chrome/browser/ssl/security_state_model_android.h"
 
 #include "base/logging.h"
-#include "chrome/browser/ssl/security_state_model.h"
+#include "chrome/browser/ssl/chrome_security_state_model_delegate.h"
 #include "content/public/browser/web_contents.h"
 #include "jni/SecurityStateModel_jni.h"
 
@@ -22,10 +22,11 @@ jint GetSecurityLevelForWebContents(
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(jweb_contents);
   DCHECK(web_contents);
-  SecurityStateModel::CreateForWebContents(web_contents);
-  SecurityStateModel* model = SecurityStateModel::FromWebContents(web_contents);
-  DCHECK(model);
-  return model->GetSecurityInfo().security_level;
+  ChromeSecurityStateModelDelegate::CreateForWebContents(web_contents);
+  ChromeSecurityStateModelDelegate* model_delegate =
+      ChromeSecurityStateModelDelegate::FromWebContents(web_contents);
+  DCHECK(model_delegate);
+  return model_delegate->GetSecurityInfo().security_level;
 }
 
 // static
@@ -35,11 +36,12 @@ jboolean IsDeprecatedSHA1Present(JNIEnv* env,
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(jweb_contents);
   DCHECK(web_contents);
-  SecurityStateModel::CreateForWebContents(web_contents);
-  SecurityStateModel* model = SecurityStateModel::FromWebContents(web_contents);
-  DCHECK(model);
-  return model->GetSecurityInfo().sha1_deprecation_status !=
-         SecurityStateModel::NO_DEPRECATED_SHA1;
+  ChromeSecurityStateModelDelegate::CreateForWebContents(web_contents);
+  ChromeSecurityStateModelDelegate* model_delegate =
+      ChromeSecurityStateModelDelegate::FromWebContents(web_contents);
+  DCHECK(model_delegate);
+  return model_delegate->GetSecurityInfo().sha1_deprecation_status !=
+         security_state::NO_DEPRECATED_SHA1;
 }
 
 // static
@@ -50,11 +52,12 @@ jboolean IsPassiveMixedContentPresent(
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(jweb_contents);
   DCHECK(web_contents);
-  SecurityStateModel::CreateForWebContents(web_contents);
-  SecurityStateModel* model = SecurityStateModel::FromWebContents(web_contents);
-  DCHECK(model);
-  return model->GetSecurityInfo().mixed_content_status ==
-             SecurityStateModel::DISPLAYED_MIXED_CONTENT ||
-         model->GetSecurityInfo().mixed_content_status ==
-             SecurityStateModel::RAN_AND_DISPLAYED_MIXED_CONTENT;
+  ChromeSecurityStateModelDelegate::CreateForWebContents(web_contents);
+  ChromeSecurityStateModelDelegate* model_delegate =
+      ChromeSecurityStateModelDelegate::FromWebContents(web_contents);
+  DCHECK(model_delegate);
+  return model_delegate->GetSecurityInfo().mixed_content_status ==
+             security_state::DISPLAYED_MIXED_CONTENT ||
+         model_delegate->GetSecurityInfo().mixed_content_status ==
+             security_state::RAN_AND_DISPLAYED_MIXED_CONTENT;
 }