diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-30 17:11:03 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-30 17:11:03 +0000 |
| commit | 385e93180e0c9498df0c3996ad4e9b709d087fe6 (patch) | |
| tree | 72d927007f0901107abd14e5774643f7fa516b4a /sql | |
| parent | e2dc04b87f64b51cffe00560017b6bcee55c5fad (diff) | |
| download | chromium_src-385e93180e0c9498df0c3996ad4e9b709d087fe6.zip chromium_src-385e93180e0c9498df0c3996ad4e9b709d087fe6.tar.gz chromium_src-385e93180e0c9498df0c3996ad4e9b709d087fe6.tar.bz2 | |
Move the DownloadManager and DownloadFile unittests to content so they can live beside the code they're testing.
-moved some of the logic about renaming intermediate files into content as the tests depended on them
-made the DownloadManagerDelegate interface have empty default implementations. Darin convinced me that it's better to have delegate interfaces in the API have emptry interfaces, for tests and to make life easier for embedders.
-removed the DownloadItemModel checks; these really were duplicating the DownloadItem::GetState call
-removed the MockDownloadManagerDelegate class since it wasn't being used anymore
-removed DownloadManagerDelegate::OnResponseCompleted since it wasn't called anymore
BUG=98716
Review URL: https://chromiumcodereview.appspot.com/9224006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119683 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sql')
0 files changed, 0 insertions, 0 deletions