WebUI: Fix/suppress some existing violations of no-restricted-globals.

ui/webui/resources/js/util.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.
 
 // <include src="assert.js">
 
 /**
  * Alias for document.getElementById. Found elements must be HTMLElements.
  * @param {string} id The ID of the element to find.
  * @return {HTMLElement} The found element or null if not found.
  */
 function $(id) {
+  // Disable getElementById restriction here, since we are instructing other
+  // places to re-use the $() that is defined here.
+  /* eslint-disable no-restricted-properties */

[Dan Beam, 2017/06/02 23:01:58]

why not

var el = document.getElementById(id);  // eslist-disable-line

or any number of ways documented here that might be less crufty?
http://eslint.org/docs/user-guide/configuring

[dpapad, 2017/06/02 23:30:03]

Done. I was not aware of eslint-disable-line and eslint-disable-next-line

   var el = document.getElementById(id);
+  /* eslint-enable no-restricted-properties */
   return el ? assertInstanceof(el, HTMLElement) : null;
 }
 
 // TODO(devlin): This should return SVGElement, but closure compiler is missing
 // those externs.
 /**
  * Alias for document.getElementById. Found elements must be SVGElements.
  * @param {string} id The ID of the element to find.
  * @return {Element} The found element or null if not found.
  */
 function getSVGElement(id) {
-  var el = document.getElementById(id);
+  var el = $(id);
   return el ? assertInstanceof(el, Element) : null;
 }
 
 /**
  * Add an accessible message to the page that will be announced to
  * users who have spoken feedback on, but will be invisible to all
  * other users. It's removed right away so it doesn't clutter the DOM.
  * @param {string} msg The text to be pronounced.
  */
 function announceAccessibleMessage(msg) {
@@ skipping 495 matching lines @@
   });
 }
 
 /**
  * @param {!Event} e
  * @return {boolean} Whether a modifier key was down when processing |e|.
  */
 function hasKeyModifiers(e) {
   return !!(e.altKey || e.ctrlKey || e.metaKey || e.shiftKey);
 }