Refactor connection_security into SecurityStateModel

content/public/browser/web_contents_delegate.h

Index: content/public/browser/web_contents_delegate.h
diff --git a/content/public/browser/web_contents_delegate.h b/content/public/browser/web_contents_delegate.h
index 5f17bc97fa9b596f1a6336c1f93bdc55aa1d1da4..bf62578aacc899654d508d269c8e1093cfff3519 100644
--- a/content/public/browser/web_contents_delegate.h
+++ b/content/public/browser/web_contents_delegate.h
@@ -92,7 +92,7 @@ class CONTENT_EXPORT WebContentsDelegate {
 
   // Called to inform the delegate that the WebContent's visible SSL state (as
   // defined by SSLStatus) changed.
-  virtual void VisibleSSLStateChanged(const WebContents* source) {}
+  virtual void VisibleSSLStateChanged(WebContents* source) {}

[Avi (use Gerrit), 2015/09/03 22:57:36]

I'm OK with this, though if you move the SSM creation to the tab helpers, then
you probably wouldn't need this change.

[estark, 2015/09/03 23:16:40]

I think I still do need it. I believe Browser::VisibleSSLStateChanged needs a
non-const WebContents in order to get a non-const SSM in order to call
SecurityStateChanged() on it.

[Avi (use Gerrit), 2015/09/03 23:21:11]

Hmmm. Yep, that's right.

 
   // Creates a new tab with the already-created WebContents 'new_contents'.
   // The window for the added contents should be reparented correctly when this