WebUI: Enable ESLint rule for missing semicolons.

chrome/browser/ui/webui/options/certificate_manager_browsertest.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.
 
 // Mac and Windows go to native certificate manager, and certificate manager
 // isn't implemented if OpenSSL is used.
 GEN('#if defined(USE_NSS_CERTS)');
 
 GEN_INCLUDE(['options_browsertest_base.js']);
 
@@ skipping 230 matching lines @@
                                'extractable': true,
                                'policy': true },
                              { 'id': 'ca_cert2',
                                'name': 'ca_cert2',
                                'readonly': false,
                                'untrusted': false,
                                'extractable': true,
                                'policy': false }],
                }],
            ]
-          ].forEach(CertificateManager.onPopulateTree)}));
+          ].forEach(CertificateManager.onPopulateTree);}));

[dschuyler, 2017/05/26 19:00:25]

optional: space after ;

[dpapad, 2017/05/26 19:08:39]

Similar to previous comment. clang-format will handle the whitespace
distribution, as follows:

 ]].forEach(CertificateManager.onPopulateTree);
}));

I prefer not to mix "eslint --fix" and "clang-format" in the same CL. It will be
addressed if/when we decide to run clang-format across our JS codebase.

[michaelpg, 2017/05/26 23:37:34]

+1, keeping them separate should make reviews and rebases easier.

   },
 };
 
 /**
  * TestFixture for testing certificate manager WebUI in the Settings page.
  * @extends {CertificateManagerWebUITest}
  * @constructor
  */
 function CertificateManagerSettingsWebUITest() {}
 
@@ skipping 134 matching lines @@
   caCertsTab.querySelector('div.tree-item').click();
   var certs = caCertsTab.querySelectorAll('div.tree-item div.tree-item');
 
   // There should be exactly three certificates displayed.
   expectEquals(certs.length, 3);
 });
 
 GEN('#endif  // defined(OS_CHROMEOS)');
 
 GEN('#endif  // defined(USE_NSS_CERTS)');