Pull SecurityStateModel out into a component

chrome/browser/ssl/chrome_security_state_model_client.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SSL_CHROME_SECURITY_STATE_MODEL_CLIENT_H_
 #define CHROME_BROWSER_SSL_CHROME_SECURITY_STATE_MODEL_CLIENT_H_
 
 #include "base/macros.h"
-#include "chrome/browser/ssl/security_state_model.h"
-#include "chrome/browser/ssl/security_state_model_client.h"
+#include "components/security_state/security_state_model.h"
+#include "components/security_state/security_state_model_client.h"
 #include "content/public/browser/web_contents_user_data.h"
 
 namespace content {
 class WebContents;
 }  // namespace content
 
 // Uses a WebContents to provide a SecurityStateModel with the
 // information that it needs to determine the page's security status.
 class ChromeSecurityStateModelClient
-    : public SecurityStateModelClient,
+    : public security_state::SecurityStateModelClient,
       public content::WebContentsUserData<ChromeSecurityStateModelClient> {
  public:
   ~ChromeSecurityStateModelClient() override;
 
-  const SecurityStateModel::SecurityInfo& GetSecurityInfo() const;
+  const security_state::SecurityStateModel::SecurityInfo& GetSecurityInfo()

[felt, 2015/12/22 17:29:08]

nit: this is repeated often enough that it might merit a `using
security_state::SecurityStateModel`

same comment throughout the files

[tfarina, 2015/12/22 17:30:30]

certainly, you won't be doing this in a header file.
maybe on the source files though.

[felt, 2015/12/22 17:34:30]

sorry, i meant to make this comment on the next (.cc) file :)

[estark, 2015/12/22 18:31:12]

Done in .cc files where it's used more than a handful of times.

+      const;
 
   // SecurityStateModelClient:
   void GetVisibleSecurityState(
-      SecurityStateModel::VisibleSecurityState* state) override;
+      security_state::SecurityStateModel::VisibleSecurityState* state) override;
   bool RetrieveCert(scoped_refptr<net::X509Certificate>* cert) 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_;
-  scoped_ptr<SecurityStateModel> security_state_model_;
+  scoped_ptr<security_state::SecurityStateModel> security_state_model_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeSecurityStateModelClient);
 };
 
 #endif  // CHROME_BROWSER_SSL_CHROME_SECURITY_STATE_MODEL_CLIENT_H_