Added download error descriptions to tooltips for Mac & Linux.

chrome/browser/download/download_item_model.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_MODEL_H_
 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_MODEL_H_
 #pragma once
 
 #include <string>
 
@@ skipping 14 matching lines @@
 // This class is an abstraction for common UI tasks associated with a download.
 class BaseDownloadItemModel {
  public:
   explicit BaseDownloadItemModel(content::DownloadItem* download)
       : download_(download) { }
   virtual ~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;
 
   // Rough percent complete. Returns -1 if the progress is unknown.
   virtual int PercentComplete() const = 0;
 
   // Get the warning text to display for a dangerous download. The |base_width|
   // is the maximum width of an embedded filename (if there is one). The metrics
   // for the filename will be based on |font|. Should only be called if
   // IsDangerous() is true.
@@ skipping 22 matching lines @@
 };
 
 // Concrete implementation of BaseDownloadItemModel.
 class DownloadItemModel : public BaseDownloadItemModel {
  public:
   explicit DownloadItemModel(content::DownloadItem* download);
   virtual ~DownloadItemModel() { }
 
   // 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,
                                   int base_width) OVERRIDE;
   virtual string16 GetWarningConfirmButtonText() OVERRIDE;
   virtual bool IsMalicious() OVERRIDE;
   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);
 };
 
 #endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_MODEL_H_