Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(488)

Unified Diff: chrome/browser/resources/shared/js/cr/ui/table/table_list.js

Issue 7606029: WebUI TaskManager: Supports background-page mode. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/shared/js/cr/ui/table/table_list.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/table/table_list.js b/chrome/browser/resources/shared/js/cr/ui/table/table_list.js
index 2d676ac4242c715ab7e6d6be3646027f3abf015b..0967a1cca638b9987b224c1155dcb5a7714a196a 100644
--- a/chrome/browser/resources/shared/js/cr/ui/table/table_list.js
+++ b/chrome/browser/resources/shared/js/cr/ui/table/table_list.js
@@ -67,17 +67,21 @@ cr.define('cr.ui.table', function() {
* @return {!ListItem} The newly created list item.
*/
createItem: function(dataItem) {
- var cm = this.table_.columnModel;
+ return this.table_.getRenderFunction().call(null, dataItem, this.table_);
+ },
+
+ renderFunction_: function(dataItem, table) {
xiyuan 2011/08/11 17:14:13 nit: prefer table as the first arg.
+ var cm = table.columnModel;
var listItem = new ListItem({label: ''});
listItem.className = 'table-row';
for (var i = 0; i < cm.size; i++) {
- var cell = this.ownerDocument.createElement('div');
+ var cell = table.ownerDocument.createElement('div');
cell.style.width = cm.getWidth(i) + '%';
cell.className = 'table-row-cell';
cell.appendChild(
- cm.getRenderFunction(i).call(null, dataItem, cm.getId(i), this));
+ cm.getRenderFunction(i).call(null, dataItem, cm.getId(i), table));
listItem.appendChild(cell);
}

Powered by Google App Engine
This is Rietveld 408576698