From 857426ccb3e0aa63433be5cf77b16e85b915281a Mon Sep 17 00:00:00 2001 From: "phajdan.jr@chromium.org" Date: Wed, 5 Aug 2009 19:40:18 +0000 Subject: Clean up SavePage UI test, remove dead code. As most of the tests moved to the browser test, this is now unneeded. TEST=none BUG=none Review URL: http://codereview.chromium.org/160666 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22511 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/download/save_page_uitest.cc | 23 ++--------------------- 1 file changed, 2 insertions(+), 21 deletions(-) (limited to 'chrome/browser/download') diff --git a/chrome/browser/download/save_page_uitest.cc b/chrome/browser/download/save_page_uitest.cc index f78fdf4..0e49b46 100644 --- a/chrome/browser/download/save_page_uitest.cc +++ b/chrome/browser/download/save_page_uitest.cc @@ -30,8 +30,7 @@ class SavePageTest : public UITest { SavePageTest() : UITest() {} void CheckFile(const FilePath& client_file, - const FilePath& server_file, - bool check_equal) { + const FilePath& server_file) { file_util::FileInfo previous, current; bool exist = false; for (int i = 0; i < 20; ++i) { @@ -47,23 +46,6 @@ class SavePageTest : public UITest { PlatformThread::Sleep(sleep_timeout_ms()); } EXPECT_TRUE(exist); - - if (check_equal) { - FilePath server_file_name; - ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, - &server_file_name)); - server_file_name = server_file_name.AppendASCII(kTestDir) - .Append(server_file); - ASSERT_TRUE(file_util::PathExists(server_file_name)); - - int64 client_file_size = 0; - int64 server_file_size = 0; - EXPECT_TRUE(file_util::GetFileSize(client_file, &client_file_size)); - EXPECT_TRUE(file_util::GetFileSize(server_file_name, &server_file_size)); - EXPECT_EQ(client_file_size, server_file_size); - EXPECT_TRUE(file_util::ContentsEqual(client_file, server_file_name)); - } - EXPECT_TRUE(file_util::DieFileDie(client_file, false)); } @@ -107,8 +89,7 @@ TEST_F(SavePageTest, CleanFilenameFromPageTitle) { EXPECT_TRUE(WaitForDownloadShelfVisible(browser.get())); automation()->SavePackageShouldPromptUser(true); - CheckFile(full_file_name, FilePath::FromWStringHack(UTF8ToWide(file_name)), - false); + CheckFile(full_file_name, FilePath::FromWStringHack(UTF8ToWide(file_name))); EXPECT_TRUE(file_util::DieFileDie(full_file_name, false)); EXPECT_TRUE(file_util::DieFileDie(dir, true)); } -- cgit v1.1