Added histograms to https://codereview.chromium.org/207553008/.

chrome/renderer/resources/neterror.js

 // Copyright 2013 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.
 
 function toggleHelpBox() {
   var helpBoxOuter = document.getElementById('help-box-outer');
   helpBoxOuter.classList.toggle('hidden');
   var moreLessButton = document.getElementById('more-less-button');
   if (helpBoxOuter.classList.contains('hidden')) {
     moreLessButton.innerText = moreLessButton.moreText;
@@ skipping 60 matching lines @@
   }
 }
 
 function loadStaleButtonClick() {
   if (window.errorPageController) {
     errorPageController.loadStaleButtonClick();
   }
 }
 
 function moreButtonClick() {
-  if (window.errorPageController) {
+  if (window.errorPageController)

[Nico, 2014/04/23 19:15:30]

This change is inconsistent with loadStaleButtonClick a few lines above, and the
other functions in this file. I'd revert the change to this file.

[Randy Smith (Not in Mondays), 2014/04/28 16:35:59]

Done.

[jar (doing other things), 2014/04/28 17:55:09]

FWIW: The change is consistent with existing lines 27, 43, and 50.  

line 75 appears to be the only example using the curlies on a one-line
statement... and should IMO probably not have the curlies.

     errorPageController.moreButtonClick();
-  }
 }
 
 <if expr="is_macosx or is_ios or is_linux or is_android">
 // Re-orders buttons. Used on Mac, Linux, and Android, where reload should go
 // on the right.
 function swapButtonOrder() {
   var reloadButton = document.getElementById('reload-button');
   var moreLessButton = document.getElementById('more-less-button');
   var staleLoadButton = document.getElementById('stale-load-button');
   reloadButton.parentNode.insertBefore(moreLessButton, reloadButton);
   reloadButton.parentNode.insertBefore(staleLoadButton, reloadButton)
 }
 document.addEventListener("DOMContentLoaded", swapButtonOrder);
 </if>