Finch experiments on SSL, malware, and phishing interstitials

chrome/browser/resources/safe_browsing/malware_block_v2.js

 // Copyright (c) 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.
 
 /**
  * Sends a command message to SafeBrowsingBlockingPage::CommandReceived.
  * @param {string} cmd The command to send.
  */
 function sendCommand(cmd) {
   window.domAutomationController.setAutomationId(1);
@@ skipping 20 matching lines @@
     $('see-less-text').hidden = false;
     $('see-more-contents').hidden = false;
     sendCommand('expandedSeeMore');
   } else {
     $('see-more-text').hidden = false;
     $('see-less-text').hidden = true;
     $('see-more-contents').hidden = true;
   }
 }
 
+function setupInterstitialExperiment() {

[James Hawkins, 2013/05/03 18:31:55]

nit: Document method.

[felt, 2013/05/03 18:43:25]

Done.

+  var condition = templateData.trialType;
+  if (condition == '' || condition == 'Default') {
+    return;  // This is the most common case.
+  } else if (condition == 'cond2MalwareNoBrand' ||
+             condition == 'cond4PhishingNoBrand') {
+    $('logo').style.display = 'none';

[James Hawkins, 2013/05/03 18:31:55]

Why display none instead of hidden?

[felt, 2013/05/03 18:43:25]

Hidden would leave extra empty whitespace, right? Display none looks nicer.

+  } else if (condition == 'cond5MalwareOneStep' ||
+             condition == 'cond6PhishingOneStep') {
+    $('see-more-contents').hidden = false;
+    $('see-less-text').hidden = true;
+    $('see-more-text').hidden = true;
+  }
+}
+
 /**
  * Onload listener to initialize javascript handlers.
  */
 document.addEventListener('DOMContentLoaded', function() {
   $('proceed-span').hidden = templateData.proceedDisabled;
 
   $('back').onclick = function() {
     sendCommand('takeMeBack');
   };
   $('proceed').onclick = function(e) {
@@ skipping 13 matching lines @@
     // preventDefaultOnPoundLinkClicks doesn't work for this link since it
     // contains <span>s, which confuse preventDefaultOnPoundLinkClicks.
     e.preventDefault();
   };
   $('check-report').onclick = savePreference;
 
   // All the links are handled by javascript sending commands back to the C++
   // handler, we don't want the default actions.
   preventDefaultOnPoundLinkClicks();  // From webui/js/util.js.
 
+  setupInterstitialExperiment();
+
   // Allow jsdisplay elements to be visible now.
   document.documentElement.classList.remove('loading');
 });