Commit ce014281 authored by yoshiki@chromium.org's avatar yoshiki@chromium.org

WebUI TaskManager: Make sort rows including "N/A" correctly.

Fixed the problem that sort didn't work correctly when some rows are "N/A". 

BUG=none 
TEST=manually


Review URL: http://codereview.chromium.org/8486014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110498 0039d316-1c4b-4281-b951-d872f2087c98
parent 67e881a0
......@@ -193,8 +193,11 @@ TaskManager.prototype = {
var compare_func = function() {
var value_id = column_id + 'Value';
return function(a, b) {
return dm.defaultValuesCompareFunction(a[value_id][0],
b[value_id][0]);
var aValues = a[value_id];
var bValues = b[value_id];
var aValue = aValues && aValues[0] || 0;
var bvalue = bValues && bValues[0] || 0;
return dm.defaultValuesCompareFunction(aValue, bvalue);
};
}();
dm.setCompareFunction(column_id, compare_func);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment