From 66ff7356a381d6c8c6c3272b095938408c4f3188 Mon Sep 17 00:00:00 2001 From: "estade@chromium.org" Date: Thu, 15 Oct 2009 05:09:50 +0000 Subject: Re-try r29078: Remove some deprecated file_util wstring functions. With the previous patch, the try bots failed with mysterious messages, so I ignored them, patched it into my windows box and tested it there manually, and found no problems. As it turns out, the try failures were real :(. But nsylvain and I found the problem: the behavior of file_util::GetDirectoryFromPath() differs from DirName() when the path is empty (officially, GetDirectoryFromPath is not supposed to support non-absolute paths, but that is not enforced). Here is a green win try result: http://build.chromium.org/buildbot/try-server/builders/win/builds/3705 mac: http://build.chromium.org/buildbot/try-server/builders/mac/builds/3491 linux: http://build.chromium.org/buildbot/try-server/builders/linux/builds/3466 I also applied this patch locally in Windows to test that it doesn't break the chrome frame compile or tests, since that's not covered by the trybots yet. Review URL: http://codereview.chromium.org/271099 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29094 0039d316-1c4b-4281-b951-d872f2087c98 --- base/file_util.h | 8 -------- 1 file changed, 8 deletions(-) (limited to 'base/file_util.h') diff --git a/base/file_util.h b/base/file_util.h index 93fdd4e..9a590a4 100644 --- a/base/file_util.h +++ b/base/file_util.h @@ -128,8 +128,6 @@ bool Delete(const std::wstring& path, bool recursive); // on different volumes, this will attempt to copy and delete. Returns // true for success. bool Move(const FilePath& from_path, const FilePath& to_path); -// Deprecated temporary compatibility function. -bool Move(const std::wstring& from_path, const std::wstring& to_path); // Renames file |from_path| to |to_path|. Both paths must be on the same // volume, or the function will fail. Destination file will be created @@ -140,8 +138,6 @@ bool ReplaceFile(const FilePath& from_path, const FilePath& to_path); // Copies a single file. Use CopyDirectory to copy directories. bool CopyFile(const FilePath& from_path, const FilePath& to_path); -// Deprecated temporary compatibility function. -bool CopyFile(const std::wstring& from_path, const std::wstring& to_path); // Copies the given path, and optionally all subdirectories and their contents // as well. @@ -159,16 +155,12 @@ bool CopyDirectory(const std::wstring& from_path, const std::wstring& to_path, // Returns true if the given path exists on the local filesystem, // false otherwise. bool PathExists(const FilePath& path); -// Deprecated temporary compatibility function. -bool PathExists(const std::wstring& path); // Returns true if the given path is writable by the user, false otherwise. bool PathIsWritable(const FilePath& path); // Returns true if the given path exists and is a directory, false otherwise. bool DirectoryExists(const FilePath& path); -// Deprecated temporary compatibility function. -bool DirectoryExists(const std::wstring& path); #if defined(OS_WIN) // Gets the creation time of the given file (expressed in the local timezone), -- cgit v1.1