Compile chrome://settings, part 9: yet another final battle

chrome/browser/resources/options/controlled_setting.js

Index: chrome/browser/resources/options/controlled_setting.js
diff --git a/chrome/browser/resources/options/controlled_setting.js b/chrome/browser/resources/options/controlled_setting.js
index df428faea98fc5c99c450b330199696aa81e60be..e75c4134920ffe51975f610b300a4bf08cfbf01d 100644
--- a/chrome/browser/resources/options/controlled_setting.js
+++ b/chrome/browser/resources/options/controlled_setting.js
@@ -55,6 +55,8 @@ cr.define('options', function() {
      * Handle changes to the associated pref by hiding any currently visible
      * bubble and updating the controlledBy property.
      * @param {Event} event Pref change event.
+     * @suppress {checkTypes}
+     * TODO(vitalyp): remove the suppression.

[Dan Beam, 2014/10/01 23:25:45]

why?

[Vitaly Pavlenko, 2014/10/01 23:38:17]

this.controlledBy defined as cr.defineProperty(), so compiler thinks it's a
string. But in fact, it's a nullable string, and it needs some refactoring to
make it be non-nullable.

Do you think we should do that right now?

      */
     handlePrefChange: function(event) {
       PageManager.hideBubble();