Separate out some more ICU from base and into base/i18n.

This moves string_util_icu. I moved the number formatting function into
base/i18n/number_formatting and just removed the other function in
string_util_icu which was TrimWhitespaceUTF8. It is only used in a few places
and isn't actually helpful (and the fact that it round-trips through UTF-16 is
better for the caller to see).

This takes out the sorting from the FileEnumerator. The comment says the
sorting is not guaranteed. I moved it into file_util_icu as a standalone
function for callers of FileEnumerator to call manually if they need sorted
results. I modified the directory lister to use this sorting instead, and filed
a bug on doing more optimal JS-based sorting.

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/267001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28405 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 3 deletions

diff --git a/chrome/browser/task_manager.h b/chrome/browser/task_manager.h
index ff402a4..a4ece90 100644
--- a/chrome/browser/task_manager.h
+++ b/chrome/browser/task_manager.h
@@ -315,9 +315,9 @@ class TaskManagerModel : public URLRequestJobTracker::JobObserver,
                                 base::ProcessMetrics** proc_metrics1,
                                 base::ProcessMetrics** proc_metrics2) const;
 
-  // Given a string containing a number, this function returns the formatted
-  // string that should be displayed in the task manager's memory cell.
-  std::wstring GetMemCellText(std::wstring* number) const;
+  // Given a number, this function returns the formatted string that should be
+  // displayed in the task manager's memory cell.
+  std::wstring GetMemCellText(int64 number) const;
 
   // The list of providers to the task manager. They are ref counted.
   ResourceProviderList providers_;