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

Side by Side Diff: chrome/browser/download/download_item_unittest.cc

Issue 8404049: Added member data to classes to support download resumption. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Incorporated Randy's comments. Created 9 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/message_loop.h" 5 #include "base/message_loop.h"
6 #include "chrome/test/base/testing_profile.h" 6 #include "chrome/test/base/testing_profile.h"
7 #include "content/browser/download/download_create_info.h" 7 #include "content/browser/download/download_create_info.h"
8 #include "content/browser/download/download_id.h" 8 #include "content/browser/download/download_id.h"
9 #include "content/browser/download/download_id_factory.h" 9 #include "content/browser/download/download_id_factory.h"
10 #include "content/browser/download/download_item.h" 10 #include "content/browser/download/download_item.h"
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 // void ShowDownloadInShell(); 117 // void ShowDownloadInShell();
118 // void CompleteDelayedDownload(); 118 // void CompleteDelayedDownload();
119 // void OnDownloadCompleting(DownloadFileManager* file_manager); 119 // void OnDownloadCompleting(DownloadFileManager* file_manager);
120 // void OnDownloadRenamedToFinalName(const FilePath& full_path); 120 // void OnDownloadRenamedToFinalName(const FilePath& full_path);
121 // set_* mutators 121 // set_* mutators
122 122
123 TEST_F(DownloadItemTest, NotificationAfterUpdate) { 123 TEST_F(DownloadItemTest, NotificationAfterUpdate) {
124 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 124 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
125 MockObserver observer(item); 125 MockObserver observer(item);
126 126
127 item->Update(kDownloadChunkSize); 127 item->Update(kDownloadChunkSize, DownloadItem::kEmptyFileHash);
128 ASSERT_TRUE(observer.CheckUpdated()); 128 ASSERT_TRUE(observer.CheckUpdated());
129 } 129 }
130 130
131 TEST_F(DownloadItemTest, NotificationAfterCancel) { 131 TEST_F(DownloadItemTest, NotificationAfterCancel) {
132 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS); 132 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS);
133 MockObserver observer1(user_cancel); 133 MockObserver observer1(user_cancel);
134 134
135 user_cancel->Cancel(true); 135 user_cancel->Cancel(true);
136 ASSERT_TRUE(observer1.CheckUpdated()); 136 ASSERT_TRUE(observer1.CheckUpdated());
137 137
(...skipping 21 matching lines...) Expand all
159 MockObserver observer(item); 159 MockObserver observer(item);
160 160
161 item->OnDownloadedFileRemoved(); 161 item->OnDownloadedFileRemoved();
162 ASSERT_TRUE(observer.CheckUpdated()); 162 ASSERT_TRUE(observer.CheckUpdated());
163 } 163 }
164 164
165 TEST_F(DownloadItemTest, NotificationAfterInterrupted) { 165 TEST_F(DownloadItemTest, NotificationAfterInterrupted) {
166 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 166 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
167 MockObserver observer(item); 167 MockObserver observer(item);
168 168
169 item->Interrupted(kDownloadChunkSize, DOWNLOAD_INTERRUPT_REASON_NONE); 169 item->Interrupted(kDownloadChunkSize,
170 DownloadItem::kEmptyFileHash,
171 DOWNLOAD_INTERRUPT_REASON_NONE);
170 ASSERT_TRUE(observer.CheckUpdated()); 172 ASSERT_TRUE(observer.CheckUpdated());
171 } 173 }
172 174
173 TEST_F(DownloadItemTest, NotificationAfterDelete) { 175 TEST_F(DownloadItemTest, NotificationAfterDelete) {
174 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 176 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
175 MockObserver observer(item); 177 MockObserver observer(item);
176 178
177 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN); 179 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN);
178 ASSERT_TRUE(observer.CheckUpdated()); 180 ASSERT_TRUE(observer.CheckUpdated());
179 } 181 }
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
244 TEST_F(DownloadItemTest, NotificationAfterTogglePause) { 246 TEST_F(DownloadItemTest, NotificationAfterTogglePause) {
245 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 247 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
246 MockObserver observer(item); 248 MockObserver observer(item);
247 249
248 item->TogglePause(); 250 item->TogglePause();
249 ASSERT_TRUE(observer.CheckUpdated()); 251 ASSERT_TRUE(observer.CheckUpdated());
250 252
251 item->TogglePause(); 253 item->TogglePause();
252 ASSERT_TRUE(observer.CheckUpdated()); 254 ASSERT_TRUE(observer.CheckUpdated());
253 } 255 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698