Add checkbox for reporting invalid TLS/SSL cert chains

chrome/browser/resources/security_warnings/safe_browsing.js

 // Copyright 2014 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.
 
 // Must match the commands handled by SafeBrowsingBlockingPage::CommandReceived.
 var SB_CMD_DO_REPORT = 'doReport';
 var SB_CMD_DONT_REPORT = 'dontReport';
 var SB_CMD_EXPANDED_SEE_MORE = 'expandedSeeMore';
 var SB_CMD_LEARN_MORE_2 = 'learnMore2';
 var SB_CMD_PROCEED = 'proceed';
 var SB_CMD_SHOW_DIAGNOSTIC = 'showDiagnostic';
 var SB_CMD_SHOW_PRIVACY = 'showPrivacy';
 var SB_CMD_TAKE_ME_BACK = 'takeMeBack';
 
-// Other constants defined in safe_browsing_blocking_page.cc.
+// Must match SSLBlockingPageCommands in ssl_blocking_page.h.
+var SSL_CMD_DO_REPORT = 6;
+var SSL_CMD_DONT_REPORT = 7;
+
+// Other constants defined in security_interstitial_page.h.
 var SB_BOX_CHECKED = 'boxchecked';
 var SB_DISPLAY_CHECK_BOX = 'displaycheckbox';
 
-// This sets up the Extended Safe Browsing Reporting opt-in.
+// This sets up the Extended Safe Browsing Reporting opt-in, either for
+// reporting malware or invalid certificate chains. Does nothing if the
+// interstitial type is not SAFEBROWSING or SSL.
 function setupCheckbox() {
-  if (loadTimeData.getString('type') != 'SAFEBROWSING' ||
-      !loadTimeData.getBoolean(SB_DISPLAY_CHECK_BOX)) {
+  var interstitialType = loadTimeData.getString('type');
+  if (interstitialType != 'SAFEBROWSING' && interstitialType != 'SSL') {
     return;
   }
 
+  if (!loadTimeData.getBoolean(SB_DISPLAY_CHECK_BOX)) {
+    return;
+  }
+
   $('opt-in-label').innerHTML = loadTimeData.getString('optInLink');
   $('opt-in-checkbox').checked = loadTimeData.getBoolean(SB_BOX_CHECKED);
   $('malware-opt-in').classList.remove('hidden');
   $('body').classList.add('safe-browsing-has-checkbox');
 
   $('opt-in-checkbox').addEventListener('click', function() {
-    sendCommand(
-        $('opt-in-checkbox').checked ? SB_CMD_DO_REPORT : SB_CMD_DONT_REPORT);
+    var command;
+    if ($('opt-in-checkbox').checked) {
+      command = interstitialType == 'SAFEBROWSING' ?
+        SB_CMD_DO_REPORT : SSL_CMD_DO_REPORT;

[felt, 2015/02/24 01:57:53]

oof, these commands need to be unified in a future clean-up CL. driving me nuts
that SB and SSL have separate commands. (note: not expecting you to do that,
just noting to myself that this needs a refactor.)

[estark, 2015/02/24 18:47:09]

noted!

+    } else {
+      command = interstitialType == 'SAFEBROWSING' ?
+        SB_CMD_DONT_REPORT : SSL_CMD_DONT_REPORT;
+    }
+    sendCommand(command);
   });
 }