Screen reader now reads the file manager selection.

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

Index: chrome/browser/resources/shared/js/cr/ui/table.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/table.js b/chrome/browser/resources/shared/js/cr/ui/table.js
index 8c207fda23f4fe8d1bb5f7dbd4169599e4b32b44..1ee0424a2e7c71bf4cf27135598e297441c1de55 100644
--- a/chrome/browser/resources/shared/js/cr/ui/table.js
+++ b/chrome/browser/resources/shared/js/cr/ui/table.js
@@ -168,17 +168,17 @@ cr.define('cr.ui', function() {
      */
     decorate: function() {
       this.list_ = this.ownerDocument.createElement('list');
+      this.appendChild(this.list_);
       TableList.decorate(this.list_);
       this.list_.selectionModel = new ListSelectionModel(this);
       this.list_.table = this;
 
       this.header_ = this.ownerDocument.createElement('div');
+      this.appendChild(this.header_);

[James Hawkins, 2012/10/01 02:12:56]

Why did you make this change?

[mtomasz, 2012/10/01 02:50:55]

We have to append the list to the DOM tree before we decorate it, because list
decorate method traverses DOM tree to find the first ancestor with set id.

       TableHeader.decorate(this.header_);
       this.header_.table = this;
 
       this.classList.add('table');
-      this.appendChild(this.header_);
-      this.appendChild(this.list_);
       this.ownerDocument.defaultView.addEventListener(
           'resize', this.header_.updateWidth.bind(this.header_));