diff options
author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-05 19:40:18 +0000 |
---|---|---|
committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-05 19:40:18 +0000 |
commit | 857426ccb3e0aa63433be5cf77b16e85b915281a (patch) | |
tree | 22c14c118f8d81ec430cfb65fe6896ad224f9169 /chrome | |
parent | e14a9f93c4ca927334008eb83127e1fb7598fe1c (diff) | |
download | chromium_src-857426ccb3e0aa63433be5cf77b16e85b915281a.zip chromium_src-857426ccb3e0aa63433be5cf77b16e85b915281a.tar.gz chromium_src-857426ccb3e0aa63433be5cf77b16e85b915281a.tar.bz2 |
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
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/download/save_page_uitest.cc | 23 |
1 files changed, 2 insertions, 21 deletions
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)); } |