summaryrefslogtreecommitdiffstats
path: root/chrome
diff options
context:
space:
mode:
authorphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-05 19:40:18 +0000
committerphajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-08-05 19:40:18 +0000
commit857426ccb3e0aa63433be5cf77b16e85b915281a (patch)
tree22c14c118f8d81ec430cfb65fe6896ad224f9169 /chrome
parente14a9f93c4ca927334008eb83127e1fb7598fe1c (diff)
downloadchromium_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.cc23
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));
}