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

Unified Diff: chrome/browser/download/download_status_updater_unittest.cc

Issue 10827207: Mountain Lion: use the system download progress. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: v3.1 Created 8 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/download/download_status_updater_unittest.cc
diff --git a/chrome/browser/download/download_status_updater_unittest.cc b/chrome/browser/download/download_status_updater_unittest.cc
index 6cc7ec03cbf801bbe5a9008956d5b8f1eb3f7176..e49b3c9b69406dfd3ba7308512ba1eda82b0967c 100644
--- a/chrome/browser/download/download_status_updater_unittest.cc
+++ b/chrome/browser/download/download_status_updater_unittest.cc
@@ -23,7 +23,8 @@ using ::testing::_;
class TestDownloadStatusUpdater : public DownloadStatusUpdater {
protected:
- virtual void UpdateAppIconDownloadProgress() OVERRIDE {
+ virtual void UpdateAppIconDownloadProgress(
+ content::DownloadItem* download) OVERRIDE {
Randy Smith (Not in Mondays) 2012/08/10 18:38:44 Shouldn't you add a test to make sure the right do
return;
}
};
@@ -120,7 +121,7 @@ class DownloadStatusUpdaterTest : public testing::Test {
}
// Set return values relevant to |DownloadStatusUpdater::GetProgress()|
- // for the specified item
+ // for the specified item.
void SetItemValues(int manager_index, int item_index,
int received_bytes, int total_bytes) {
EXPECT_CALL(*Item(manager_index, item_index), GetReceivedBytes())

Powered by Google App Engine
This is Rietveld 408576698