Add constructor to create SSLStatus from SSLInfo

chrome/browser/ssl/ssl_blocking_page.cc

Index: chrome/browser/ssl/ssl_blocking_page.cc
diff --git a/chrome/browser/ssl/ssl_blocking_page.cc b/chrome/browser/ssl/ssl_blocking_page.cc
index 97148f38d5d3a178c6453f327218e67e353500fe..c072c83a1c2ac0a0388e23fc5cf8c627d960a951 100644
--- a/chrome/browser/ssl/ssl_blocking_page.cc
+++ b/chrome/browser/ssl/ssl_blocking_page.cc
@@ -299,6 +299,7 @@ void SSLBlockingPage::OverrideEntry(NavigationEntry* entry) {
   entry->GetSSL().cert_id = cert_id;
   entry->GetSSL().cert_status = ssl_info_.cert_status;
   entry->GetSSL().security_bits = ssl_info_.security_bits;
+  entry->GetSSL().connection_status = ssl_info_.connection_status;

[meacer, 2015/09/08 23:05:54]

Looks like BadClockBlockingPage doesn't set this either. Should it? In fact any
time we add a new field to SSL status we'll need to go fix all OverrideEntry
instances. WDYT about doing something like this?:

entry->GetSSL() = MakeSSLStatus(ssl_info, cert_id, AUTH_BROKEN)

[estark, 2015/09/08 23:19:58]

Ouch, good point. BadClockBlockingPage does need to set this as well.
That seems reasonable. Do you think MakeSSLStatus() should go in ssl_status.h?

 }
 
 void SSLBlockingPage::SetSSLCertReporterForTesting(