JS style nits

chrome/browser/resources/certificate_viewer.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.
 
 cr.define('cert_viewer', function() {
   'use strict';
 
   /**
    * Initialize the certificate viewer dialog by wiring up the close button,
    * substituting in translated strings and requesting certificate details.
    */
   function initialize() {
     $('export').onclick = exportCertificate;
     cr.ui.decorate('tabbox', cr.ui.TabBox);
 
     initializeTree($('hierarchy'), showCertificateFields);
     initializeTree($('cert-fields'), showCertificateFieldValue);
 
     i18nTemplate.process(document, templateData);
     stripGtkAccessorKeys();
     chrome.send('requestCertificateInfo');
     // TODO(kochi): ESC key should be handled in the views window side.
     document.addEventListener('keydown', function(e) {
       if (e.keyCode == 27) {  // ESC

[Dan Beam, 2012/04/05 22:24:13]

nit: no curlies

[Tyler Breisacher (Chromium), 2012/04/05 22:49:55]

Done.

         chrome.send('DialogClose');
       }
     });
   }
 
   /**
    * Initialize a cr.ui.Tree object from a given element using the specified
    * change handler.
    * @param {HTMLElement} tree The HTMLElement to style as a tree.
    * @param {function()} handler Function to call when a node is selected.
    */
   function initializeTree(tree, handler) {
     cr.ui.decorate(tree, cr.ui.Tree);
     tree.detail = {payload: {}, children: {}};
     tree.addEventListener('change', handler);
   }
 
   /**
    * The tab name strings in the languages file have accessor keys indicated
    * by a preceding & sign. Strip these out for now.
-   * @TODO(flackr) These accessor keys could be implemented with Javascript or
+   * TODO(flackr) These accessor keys could be implemented with Javascript or
    *     translated strings could be added / modified to remove the & sign.
    */
   function stripGtkAccessorKeys() {
     // Copy all the tab labels into an array.
     var nodes = Array.prototype.slice.call($('tabs').childNodes, 0);
     nodes.push($('export'));
     for (var i = 0; i < nodes.length; i++)
-      nodes[i].textContent = nodes[i].textContent.replace('&','');
+      nodes[i].textContent = nodes[i].textContent.replace('&', '');
   }
 
   /**
    * Expand all nodes of the given tree object.
    * @param {cr.ui.Tree} tree The tree object to expand all nodes on.
    */
   function revealTree(tree) {
     tree.expanded = true;
     for (var key in tree.detail.children) {
       revealTree(tree.detail.children[key]);
@@ skipping 109 matching lines @@
   }
 
   return {
     initialize: initialize,
     getCertificateInfo: getCertificateInfo,
     getCertificateFields: getCertificateFields,
   };
 });
 
 document.addEventListener('DOMContentLoaded', cert_viewer.initialize);