Clock interstitial: refine iOS text and hide 'Advanced' button.

chrome/browser/resources/security_warnings/interstitial_v2.js

Index: chrome/browser/resources/security_warnings/interstitial_v2.js
diff --git a/chrome/browser/resources/security_warnings/interstitial_v2.js b/chrome/browser/resources/security_warnings/interstitial_v2.js
index 56ae8573a3c3f7c05b6b0aba4fd22975ff227f8a..a738f53c0fce453ead0c74087bcea54c3ccd5f4f 100644
--- a/chrome/browser/resources/security_warnings/interstitial_v2.js
+++ b/chrome/browser/resources/security_warnings/interstitial_v2.js
@@ -67,6 +67,8 @@ function setupEvents() {
   var overridable = loadTimeData.getBoolean('overridable');
   var ssl = loadTimeData.getString('type') === 'SSL';
   var badClock = ssl && loadTimeData.getBoolean('bad_clock');
+  var hidePrimaryButton = badClock && loadTimeData.getBoolean(
+                              'hide_primary_button');

[Bernhard Bauer, 2014/11/03 12:18:08]

Nit: just indent this four spaces.

[lgarron, 2014/11/04 02:13:50]

Acknowledged.

 
   if (ssl) {
     $('body').classList.add(badClock ? 'bad-clock' : 'ssl');
@@ -76,16 +78,21 @@ function setupEvents() {
     $('body').classList.add('safe-browsing');
   }
 
-  $('primary-button').addEventListener('click', function() {
-    if (!ssl)
-      sendCommand(SB_CMD_TAKE_ME_BACK);
-    else if (badClock)
-      sendCommand(SSL_CMD_CLOCK);
-    else if (overridable)
-      sendCommand(SSL_CMD_DONT_PROCEED);
-    else
-      sendCommand(SSL_CMD_RELOAD);
-  });
+  if (hidePrimaryButton) {
+    $('primary-button').classList.add('hidden');
+  }
+  else {

[felt, 2014/11/01 21:39:11]

} else {

[lgarron, 2014/11/04 02:13:50]

Acknowledged.

+    $('primary-button').addEventListener('click', function() {
+      if (!ssl)
+        sendCommand(SB_CMD_TAKE_ME_BACK);
+      else if (badClock)
+        sendCommand(SSL_CMD_CLOCK);
+      else if (overridable)
+        sendCommand(SSL_CMD_DONT_PROCEED);
+      else
+        sendCommand(SSL_CMD_RELOAD);
+    });
+  }
 
   if (overridable) {
     $('proceed-link').addEventListener('click', function(event) {
@@ -110,18 +117,14 @@ function setupEvents() {
   }
 
   $('details-button').addEventListener('click', function(event) {
-    if (badClock) {
-      sendCommand(SSL_CMD_RELOAD);
-    } else {
-      var hiddenDetails = $('details').classList.toggle('hidden');
-      $('details-button').innerText = hiddenDetails ?
-          loadTimeData.getString('openDetails') :
-          loadTimeData.getString('closeDetails');
-      if (!expandedDetails) {
-        // Record a histogram entry only the first time that details is opened.
-        sendCommand(ssl ? SSL_CMD_MORE : SB_CMD_EXPANDED_SEE_MORE);
-        expandedDetails = true;
-      }
+    var hiddenDetails = $('details').classList.toggle('hidden');
+    $('details-button').innerText = hiddenDetails ?
+        loadTimeData.getString('openDetails') :
+        loadTimeData.getString('closeDetails');
+    if (!expandedDetails) {
+      // Record a histogram entry only the first time that details is opened.
+      sendCommand(ssl ? SSL_CMD_MORE : SB_CMD_EXPANDED_SEE_MORE);
+      expandedDetails = true;
     }
   });