Refactor SecurityStateModel/Clients for simplicity and reusability.

chrome/browser/ssl/chrome_security_state_model_client.h

Index: chrome/browser/ssl/chrome_security_state_model_client.h
diff --git a/chrome/browser/ssl/chrome_security_state_model_client.h b/chrome/browser/ssl/chrome_security_state_model_client.h
index be12ac1b9c20d71addcecb9aef70b68d4a101abb..074ecb15ead0f266a6078c0ddcee81ea46f215b2 100644
--- a/chrome/browser/ssl/chrome_security_state_model_client.h
+++ b/chrome/browser/ssl/chrome_security_state_model_client.h
@@ -8,14 +8,15 @@
 #include <memory>
 
 #include "base/macros.h"
-#include "components/security_state/security_state_model.h"
-#include "components/security_state/security_state_model_client.h"
+#include "components/security_state/content/web_contents_security_state_model_client.h"
+#include "components/security_state/core/security_state_model.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 #include "third_party/WebKit/public/platform/WebSecurityStyle.h"
 
 namespace content {
 struct SecurityStyleExplanations;
+class NavigationEntry;
 class NavigationHandle;
 class WebContents;
 }  // namespace content
@@ -23,15 +24,12 @@ class WebContents;
 // Uses a WebContents to provide a SecurityStateModel with the
 // information that it needs to determine the page's security status.
 class ChromeSecurityStateModelClient
-    : public security_state::SecurityStateModelClient,
+    : public security_state::WebContentsSecurityStateModelClient,
       public content::WebContentsObserver,
       public content::WebContentsUserData<ChromeSecurityStateModelClient> {
  public:
   ~ChromeSecurityStateModelClient() override;
 
-  void GetSecurityInfo(
-      security_state::SecurityStateModel::SecurityInfo* result) const;
-
   // Called when the NavigationEntry's SSLStatus changes.
   void VisibleSSLStateChanged();
 
@@ -39,26 +37,18 @@ class ChromeSecurityStateModelClient
   void DidFinishNavigation(
       content::NavigationHandle* navigation_handle) override;
 
-  // Returns the SecurityStyle that should be applied to a WebContents
-  // with the given |security_info|. Populates
-  // |security_style_explanations| to explain why the returned
-  // SecurityStyle was chosen.
-  static blink::WebSecurityStyle GetSecurityStyle(
-      const security_state::SecurityStateModel::SecurityInfo& security_info,
-      content::SecurityStyleExplanations* security_style_explanations);
-
   // SecurityStateModelClient:
-  void GetVisibleSecurityState(
-      security_state::SecurityStateModel::VisibleSecurityState* state) override;
   bool UsedPolicyInstalledCertificate() override;
-  bool IsOriginSecure(const GURL& url) override;
 
  private:
   explicit ChromeSecurityStateModelClient(content::WebContents* web_contents);
   friend class content::WebContentsUserData<ChromeSecurityStateModelClient>;
 
-  content::WebContents* web_contents_;
-  std::unique_ptr<security_state::SecurityStateModel> security_state_model_;
+  // WebContentsSecurityStateModelClient:
+  void CheckMalwareStatus(
+      content::NavigationEntry* entry,
+      content::WebContents* web_contents,
+      security_state::SecurityStateModel::VisibleSecurityState* state) override;
 
   // True if a console has been logged about an omnibox warning that
   // will be shown in future versions of Chrome for insecure HTTP