Add SecurityStateModelClient interface and implementation

chrome/browser/ssl/security_state_model.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_SECURITY_STATE_MODEL_H_
 #define CHROME_BROWSER_SSL_SECURITY_STATE_MODEL_H_
 
 #include "base/macros.h"
 #include "content/public/browser/web_contents_user_data.h"
 #include "content/public/common/security_style.h"
 #include "content/public/common/ssl_status.h"
 #include "net/cert/cert_status_flags.h"
 #include "net/cert/sct_status_flags.h"
 #include "net/cert/x509_certificate.h"
 
 namespace content {
 class NavigationHandle;
 class WebContents;
 }  // namespace content
 
 class Profile;
+class SecurityStateModelDelegate;
 
 // SecurityStateModel provides high-level security information about a
 // page or request. It is attached to a WebContents and will provide the
 // security info for that WebContents.
 //
 // SecurityStateModel::SecurityInfo is the main data structure computed
 // by a SecurityStateModel. SecurityInfo contains a SecurityLevel (which
 // is a single value describing the overall security state) along with
 // information that a consumer might want to display in UI to explain or
 // elaborate on the SecurityLevel.
@@ skipping 95 matching lines @@
 
   // Returns a SecurityInfo describing the current page. Results are
   // cached so that computation is only done once per visible
   // NavigationEntry.
   const SecurityInfo& GetSecurityInfo() const;
 
   // Returns a SecurityInfo describing an individual request for the
   // given |profile|.
   static void SecurityInfoForRequest(const GURL& url,
                                      const content::SSLStatus& ssl,
-                                     Profile* profile,
+                                     Profile* prfoile,

[blundell, 2015/11/23 16:06:17]

nit: I liked the spelling better before ;).

[estark, 2015/11/23 16:23:39]

Doh. Done.

+                                     bool used_known_mitm_certificate,
                                      SecurityInfo* security_info);
 
  private:
   explicit SecurityStateModel(content::WebContents* web_contents);
   friend class content::WebContentsUserData<SecurityStateModel>;
 
   // The WebContents for which this class describes the security status.
   content::WebContents* web_contents_;
 
   // These data members cache the SecurityInfo for the visible
   // NavigationEntry. They are marked mutable so that the const accessor
   // GetSecurityInfo() can update the cache.
   mutable SecurityInfo security_info_;
   mutable GURL visible_url_;
   mutable content::SSLStatus visible_ssl_status_;
 
+  scoped_ptr<SecurityStateModelDelegate> delegate_;

[blundell, 2015/11/23 16:06:17]

nit: Let's add a comment here explaining how this will shortly be injected with
a TODO(estark).

[estark, 2015/11/23 16:23:40]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(SecurityStateModel);
 };
 
 #endif  // CHROME_BROWSER_SSL_SECURITY_STATE_MODEL_H_