[Chrome Settings UI] If User Exceptions are not allowed, prevent editing / viewing.

chrome/browser/resources/options/content_settings.js

Index: chrome/browser/resources/options/content_settings.js
diff --git a/chrome/browser/resources/options/content_settings.js b/chrome/browser/resources/options/content_settings.js
index 33cac98257f4c9a08e87a3fa3c3db94a921c6203..3d7a7ca39c2158ba61bcc5115553e4f9c856295b 100644
--- a/chrome/browser/resources/options/content_settings.js
+++ b/chrome/browser/resources/options/content_settings.js
@@ -131,6 +131,18 @@ cr.define('options', function() {
   };
 
   /**
+   * Updates UI to allow/disallow user from using exceptions for a particular
+   * content setting.
+   * @param {string} type The content settings type being updated.
+   * @param {boolean} allowUserExceptions Whether user may set exceptions.

[Dan Beam, 2016/05/03 04:20:25]

please closure compile your code, this name doesn't match (allowUserExceptions
vs userExceptionsAllowed) and is a boolean here but a string in the other class

[huangs, 2016/05/03 18:34:18]

Fixed.

How do you "properly" run Closure on this? I thought the code are just
Closure-like, without using closure?  Anyway, ran 

java -jar C:\closure\compiler.jar %* --checks_only --warning_level VERBOSE
--jscomp_warning "*"

to content_settings.js and content_settings_exceptions_area.js .  There are
bunch of existing errors.  Ensured that I didn't add new genuine error.

[Dan Beam, 2016/05/04 01:44:42]

https://groups.google.com/a/chromium.org/d/msg/chromium-dev/15zvJ-TjMQY/bJq1r...

+   */
+  ContentSettings.setUserExceptionsAllowed = function(type,
+                                                      userExceptionsAllowed) {
+    // Update exceptions UI window.
+    this.getExceptionsList(type, 'normal').setAllowEdit(userExceptionsAllowed);
+  };
+
+  /**
    * Initializes an exceptions list.
    * @param {string} type The content type that we are setting exceptions for.
    * @param {Array<options.Exception>} exceptions An array of pairs, where the