Added download error descriptions to tooltips for Mac & Linux.

chrome/browser/download/download_item_model.h

Index: chrome/browser/download/download_item_model.h
diff --git a/chrome/browser/download/download_item_model.h b/chrome/browser/download/download_item_model.h
index bddf911174a547b5a59554e051285e94747099c9..5c2e21b725d93e3d255f754d8d1ead3410212aaf 100644
--- a/chrome/browser/download/download_item_model.h
+++ b/chrome/browser/download/download_item_model.h
@@ -32,6 +32,9 @@ class BaseDownloadItemModel {
   // Cancel the task corresponding to the item.
   virtual void CancelTask() = 0;
 
+  // Get the progress text to display.

[asanka, 2012/04/04 17:27:04]

Consider adding some example text here.

[ahendrickson, 2012/04/04 17:49:32]

Done.

+  virtual string16 GetProgressText() = 0;
+
   // Get the status text to display.
   virtual string16 GetStatusText() = 0;
 
@@ -74,6 +77,7 @@ class DownloadItemModel : public BaseDownloadItemModel {
 
   // BaseDownloadItemModel
   virtual void CancelTask() OVERRIDE;
+  virtual string16 GetProgressText() OVERRIDE;
   virtual string16 GetStatusText() OVERRIDE;
   virtual int PercentComplete() const OVERRIDE;
   virtual string16 GetWarningText(const gfx::Font& font,
@@ -83,6 +87,11 @@ class DownloadItemModel : public BaseDownloadItemModel {
   virtual bool IsDangerous() OVERRIDE;
 
  private:
+  bool GetStatusData(int64* size_return,

[sky, 2012/04/04 16:55:34]

Add a description of this, including what the return value means.

[ahendrickson, 2012/04/04 17:49:32]

Done.

+                     int64* total_return,
+                     string16* simple_size_return,
+                     string16* simple_total_return);
+
   DISALLOW_COPY_AND_ASSIGN(DownloadItemModel);
 };