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

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, 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 "base/pickle.h"
6 #include "chrome/test/base/testing_profile.h" 7 #include "chrome/test/base/testing_profile.h"
7 #include "content/browser/download/download_create_info.h" 8 #include "content/browser/download/download_create_info.h"
8 #include "content/browser/download/download_id.h" 9 #include "content/browser/download/download_id.h"
9 #include "content/browser/download/download_id_factory.h" 10 #include "content/browser/download/download_id_factory.h"
10 #include "content/browser/download/download_item.h" 11 #include "content/browser/download/download_item.h"
11 #include "content/browser/download/download_status_updater.h" 12 #include "content/browser/download/download_status_updater.h"
12 #include "content/browser/download/interrupt_reasons.h" 13 #include "content/browser/download/interrupt_reasons.h"
13 #include "content/browser/download/mock_download_manager.h" 14 #include "content/browser/download/mock_download_manager.h"
14 #include "content/browser/download/mock_download_manager_delegate.h" 15 #include "content/browser/download/mock_download_manager_delegate.h"
15 #include "content/test/test_browser_thread.h" 16 #include "content/test/test_browser_thread.h"
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 // void ShowDownloadInShell(); 118 // void ShowDownloadInShell();
118 // void CompleteDelayedDownload(); 119 // void CompleteDelayedDownload();
119 // void OnDownloadCompleting(DownloadFileManager* file_manager); 120 // void OnDownloadCompleting(DownloadFileManager* file_manager);
120 // void OnDownloadRenamedToFinalName(const FilePath& full_path); 121 // void OnDownloadRenamedToFinalName(const FilePath& full_path);
121 // set_* mutators 122 // set_* mutators
122 123
123 TEST_F(DownloadItemTest, NotificationAfterUpdate) { 124 TEST_F(DownloadItemTest, NotificationAfterUpdate) {
124 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 125 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
125 MockObserver observer(item); 126 MockObserver observer(item);
126 127
127 item->Update(kDownloadChunkSize); 128 item->Update(kDownloadChunkSize, Pickle());
128 ASSERT_TRUE(observer.CheckUpdated()); 129 ASSERT_TRUE(observer.CheckUpdated());
129 } 130 }
130 131
131 TEST_F(DownloadItemTest, NotificationAfterCancel) { 132 TEST_F(DownloadItemTest, NotificationAfterCancel) {
132 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS); 133 DownloadItem* user_cancel = CreateDownloadItem(DownloadItem::IN_PROGRESS);
133 MockObserver observer1(user_cancel); 134 MockObserver observer1(user_cancel);
134 135
135 user_cancel->Cancel(true); 136 user_cancel->Cancel(true);
136 ASSERT_TRUE(observer1.CheckUpdated()); 137 ASSERT_TRUE(observer1.CheckUpdated());
137 138
(...skipping 21 matching lines...) Expand all
159 MockObserver observer(item); 160 MockObserver observer(item);
160 161
161 item->OnDownloadedFileRemoved(); 162 item->OnDownloadedFileRemoved();
162 ASSERT_TRUE(observer.CheckUpdated()); 163 ASSERT_TRUE(observer.CheckUpdated());
163 } 164 }
164 165
165 TEST_F(DownloadItemTest, NotificationAfterInterrupted) { 166 TEST_F(DownloadItemTest, NotificationAfterInterrupted) {
166 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 167 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
167 MockObserver observer(item); 168 MockObserver observer(item);
168 169
169 item->Interrupted(kDownloadChunkSize, DOWNLOAD_INTERRUPT_REASON_NONE); 170 item->Interrupted(kDownloadChunkSize,
171 Pickle(),
172 DOWNLOAD_INTERRUPT_REASON_NONE);
170 ASSERT_TRUE(observer.CheckUpdated()); 173 ASSERT_TRUE(observer.CheckUpdated());
171 } 174 }
172 175
173 TEST_F(DownloadItemTest, NotificationAfterDelete) { 176 TEST_F(DownloadItemTest, NotificationAfterDelete) {
174 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 177 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
175 MockObserver observer(item); 178 MockObserver observer(item);
176 179
177 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN); 180 item->Delete(DownloadItem::DELETE_DUE_TO_BROWSER_SHUTDOWN);
178 ASSERT_TRUE(observer.CheckUpdated()); 181 ASSERT_TRUE(observer.CheckUpdated());
179 } 182 }
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 TEST_F(DownloadItemTest, NotificationAfterTogglePause) { 244 TEST_F(DownloadItemTest, NotificationAfterTogglePause) {
242 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS); 245 DownloadItem* item = CreateDownloadItem(DownloadItem::IN_PROGRESS);
243 MockObserver observer(item); 246 MockObserver observer(item);
244 247
245 item->TogglePause(); 248 item->TogglePause();
246 ASSERT_TRUE(observer.CheckUpdated()); 249 ASSERT_TRUE(observer.CheckUpdated());
247 250
248 item->TogglePause(); 251 item->TogglePause();
249 ASSERT_TRUE(observer.CheckUpdated()); 252 ASSERT_TRUE(observer.CheckUpdated());
250 } 253 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698