Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(448)

Unified Diff: webrtc/rtc_base/unixfilesystem.h

Issue 2995413002: Delete Filesystem::GetTemporaryFolder. (Closed)
Patch Set: Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/rtc_base/optionsfile_unittest.cc ('k') | webrtc/rtc_base/unixfilesystem.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/rtc_base/unixfilesystem.h
diff --git a/webrtc/rtc_base/unixfilesystem.h b/webrtc/rtc_base/unixfilesystem.h
index 0ed36b43346a55b1b025b43afcdb42c41aa4b1c8..7270b83778be7df3df2d5552f82c71cdbbdb86b1 100644
--- a/webrtc/rtc_base/unixfilesystem.h
+++ b/webrtc/rtc_base/unixfilesystem.h
@@ -22,18 +22,6 @@ class UnixFilesystem : public FilesystemInterface {
UnixFilesystem();
~UnixFilesystem() override;
-#if defined(WEBRTC_ANDROID) || defined(WEBRTC_MAC)
- // Android does not have a native code API to fetch the app data or temp
- // folders. That needs to be passed into this class from Java. Similarly, iOS
- // only supports an Objective-C API for fetching the folder locations, so that
- // needs to be passed in here from Objective-C. Or at least that used to be
- // the case; now the ctor will do the work if necessary and possible.
- // TODO(fischman): add an Android version that uses JNI and drop the
- // SetApp*Folder() APIs once external users stop using them.
- static void SetAppDataFolder(const std::string& folder);
- static void SetAppTempFolder(const std::string& folder);
-#endif
-
// This will attempt to delete the file located at filename.
// It will fail with VERIY if you pass it a non-existant file, or a directory.
bool DeleteFile(const Pathname& filename) override;
@@ -65,12 +53,6 @@ class UnixFilesystem : public FilesystemInterface {
std::string TempFilename(const Pathname& dir,
const std::string& prefix) override;
- // A folder appropriate for storing temporary files (Contents are
- // automatically deleted when the program exists)
- bool GetTemporaryFolder(Pathname& path,
- bool create,
- const std::string* append) override;
-
bool GetFileSize(const Pathname& path, size_t* size) override;
private:
« no previous file with comments | « webrtc/rtc_base/optionsfile_unittest.cc ('k') | webrtc/rtc_base/unixfilesystem.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698