Add icon-visibility attribute to cr.ui.Tree.

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

Index: chrome/browser/resources/shared/js/cr/ui/tree.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/tree.js b/chrome/browser/resources/shared/js/cr/ui/tree.js
index c1386358ec03028830dca7367e17279597342881..fc2afcec27e0029dea146636020eeadb3a75bfe0 100644
--- a/chrome/browser/resources/shared/js/cr/ui/tree.js
+++ b/chrome/browser/resources/shared/js/cr/ui/tree.js
@@ -218,6 +218,15 @@ cr.define('cr.ui', function() {
   };
 
   /**
+   * Determines the visibility of icons next to the treeItem labels. If defined,
+   * space is reserved next to treeItem labels for icons. If set to 'parent',
+   * folder icons will be displayed next to expandable parent nodes. If set to
+   * 'all' folder icons will be displayed next to all nodes. Icons can be set
+   * using the treeItem's icon property.
+   */
+  cr.defineProperty(Tree, 'icon-visibility', cr.PropertyKind.ATTR);

[arv (Not doing code reviews), 2011/10/28 16:15:12]

Sorry, this will not work. The property name should be iconVisibility.

The default is probably 'all'?

[flackr, 2011/10/28 22:32:50]

Sorry, is there an appropriate place to define the default? I've looked for
similar examples and checked the cr API. As it is with this patch the default
(undefined) will behave the same as 'parent'.

[arv (Not doing code reviews), 2011/10/28 22:41:03]

I think I actually removed support for default value for attribute mapped
properties since it adds complexity.

[flackr, 2011/10/31 14:37:56]

No problem, I changed the styles to make anything other than 'hidden' or
'parent' (including nothing) behave as 'all'.

+
+  /**
    * This is used as a blueprint for new tree item elements.
    * @type {!HTMLElement}
    */