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

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: Merged with trunk Created 9 years 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
« no previous file with comments | « no previous file | chrome/browser/download/download_manager_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "base/stl_util.h" 6 #include "base/stl_util.h"
7 #include "base/threading/thread.h" 7 #include "base/threading/thread.h"
8 #include "chrome/test/base/testing_profile.h" 8 #include "chrome/test/base/testing_profile.h"
9 #include "content/browser/download/download_create_info.h" 9 #include "content/browser/download/download_create_info.h"
10 #include "content/browser/download/download_id.h" 10 #include "content/browser/download/download_id.h"
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 // void ShowDownloadInShell(); 152 // void ShowDownloadInShell();
153 // void CompleteDelayedDownload(); 153 // void CompleteDelayedDownload();
154 // void OnDownloadCompleting(DownloadFileManager* file_manager); 154 // void OnDownloadCompleting(DownloadFileManager* file_manager);
155 // void OnDownloadRenamedToFinalName(const FilePath& full_path); 155 // void OnDownloadRenamedToFinalName(const FilePath& full_path);
156 // set_* mutators 156 // set_* mutators
157 157
158 TEST_F(DownloadItemTest, NotificationAfterUpdate) { 158 TEST_F(DownloadItemTest, NotificationAfterUpdate) {
159 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 159 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
160 MockObserver observer(item); 160 MockObserver observer(item);
161 161
162 item->UpdateProgress(kDownloadChunkSize, kDownloadSpeed); 162 item->UpdateProgress(kDownloadChunkSize, kDownloadSpeed, "");
163 ASSERT_TRUE(observer.CheckUpdated()); 163 ASSERT_TRUE(observer.CheckUpdated());
164 EXPECT_EQ(kDownloadSpeed, item->CurrentSpeed()); 164 EXPECT_EQ(kDownloadSpeed, item->CurrentSpeed());
165 } 165 }
166 166
167 TEST_F(DownloadItemTest, NotificationAfterCancel) { 167 TEST_F(DownloadItemTest, NotificationAfterCancel) {
168 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS); 168 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS);
169 MockObserver observer1(user_cancel); 169 MockObserver observer1(user_cancel);
170 170
171 user_cancel->Cancel(true); 171 user_cancel->Cancel(true);
172 ASSERT_TRUE(observer1.CheckUpdated()); 172 ASSERT_TRUE(observer1.CheckUpdated());
(...skipping 22 matching lines...) Expand all
195 MockObserver observer(item); 195 MockObserver observer(item);
196 196
197 item->OnDownloadedFileRemoved(); 197 item->OnDownloadedFileRemoved();
198 ASSERT_TRUE(observer.CheckUpdated()); 198 ASSERT_TRUE(observer.CheckUpdated());
199 } 199 }
200 200
201 TEST_F(DownloadItemTest, NotificationAfterInterrupted) { 201 TEST_F(DownloadItemTest, NotificationAfterInterrupted) {
202 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 202 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
203 MockObserver observer(item); 203 MockObserver observer(item);
204 204
205 item->Interrupted(kDownloadChunkSize, DOWNLOAD_INTERRUPT_REASON_NONE); 205 item->Interrupted(kDownloadChunkSize, "", DOWNLOAD_INTERRUPT_REASON_NONE);
206 ASSERT_TRUE(observer.CheckUpdated()); 206 ASSERT_TRUE(observer.CheckUpdated());
207 } 207 }
208 208
209 TEST_F(DownloadItemTest, NotificationAfterDelete) { 209 TEST_F(DownloadItemTest, NotificationAfterDelete) {
210 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 210 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
211 MockObserver observer(item); 211 MockObserver observer(item);
212 212
213 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN); 213 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN);
214 ASSERT_TRUE(observer.CheckUpdated()); 214 ASSERT_TRUE(observer.CheckUpdated());
215 } 215 }
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
339 item->SetExternalData(&external_data_test_string, test3); 339 item->SetExternalData(&external_data_test_string, test3);
340 EXPECT_EQ(static_cast<DownloadItem::ExternalData*>(test3), 340 EXPECT_EQ(static_cast<DownloadItem::ExternalData*>(test3),
341 item->GetExternalData(&external_data_test_string)); 341 item->GetExternalData(&external_data_test_string));
342 DestroyDownloadItem(item); 342 DestroyDownloadItem(item);
343 EXPECT_EQ(3, destructor_called); 343 EXPECT_EQ(3, destructor_called);
344 } 344 }
345 345
346 TEST(MockDownloadItem, Compiles) { 346 TEST(MockDownloadItem, Compiles) {
347 MockDownloadItem mock_item; 347 MockDownloadItem mock_item;
348 } 348 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/download/download_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698