[tabbed options] more work on content settings exceptions lists

chrome/browser/resources/shared/js/cr/ui/list.js

Index: chrome/browser/resources/shared/js/cr/ui/list.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/list.js b/chrome/browser/resources/shared/js/cr/ui/list.js
index 33a984ebf5e87e3565666798f8bf9ca90a2343d6..c2e032dd5bc8c0e9d046b083e1e4fd4304e54feb 100644
--- a/chrome/browser/resources/shared/js/cr/ui/list.js
+++ b/chrome/browser/resources/shared/js/cr/ui/list.js
@@ -223,12 +223,12 @@ cr.define('cr.ui', function() {
     },
 
     /**
-     * The HTML elements representing the items. This is just all the element
+     * The HTML elements representing the items. This is just all the <li>

[arv (Not doing code reviews), 2010/12/15 02:47:27]

I don't think <li> is correct here. Is there any other place that this is hard
coded? I think instances can use any tag name they want

[Evan Stade, 2010/12/15 04:12:38]

good point

      * children but subclasses may override this to filter out certain elements.
      * @type {HTMLCollection}
      */
     get items() {

[arv (Not doing code reviews), 2010/12/15 02:47:27]

Yeah, this cannot have been used before since it was clearly broken.

-      return this.children;
+      return this.querySelectorAll('li');

[arv (Not doing code reviews), 2010/12/15 02:47:27]

This also gives nested elements. I think we need to use children and filter out
the spacers.

     },
 
     batchCount_: 0,