Expose TLS settings in the Security panel overview, and call out individual obsolete settings.

chrome/browser/ui/browser.cc

 // Copyright 2012 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.
 
 #include "chrome/browser/ui/browser.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <string>
@@ skipping 1383 matching lines @@
               security_info.cert_id));
     }
   }
 
   if (security_info.is_secure_protocol_and_ciphersuite) {
     security_style_explanations->secure_explanations.push_back(
         content::SecurityStyleExplanation(
             l10n_util::GetStringUTF8(IDS_SECURE_PROTOCOL_AND_CIPHERSUITE),
             l10n_util::GetStringUTF8(
                 IDS_SECURE_PROTOCOL_AND_CIPHERSUITE_DESCRIPTION)));
+  } else {
+    // TODO: Include cipher suite name
+    security_style_explanations->info_explanations.push_back(
+        content::SecurityStyleExplanation(
+            l10n_util::GetStringUTF8(IDS_OBSOLETE_PROTOCOL_AND_CIPHERSUITE),
+            l10n_util::GetStringUTF8(
+                IDS_OBSOLETE_PROTOCOL_AND_CIPHERSUITE_DESCRIPTION))
+  );

[estark, 2016/02/24 02:10:50]

formatting looks weird here

[lgarron, 2016/04/09 03:22:50]

The last I've moved the final ); onto the previous line.

   }
 
   return security_style;
 }
 
 void Browser::ShowCertificateViewerInDevTools(
     content::WebContents* web_contents, int cert_id) {
   DevToolsWindow* devtools_window =
       DevToolsWindow::GetInstanceForInspectedWebContents(web_contents);
   if (devtools_window)
@@ skipping 1318 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         ui::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }