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

Unified Diff: webkit/fileapi/local_file_util_unittest.cc

Issue 7470037: [Refactor] to rename and re-layer the file_util stack layers. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Rebased on the svn tree. Created 9 years, 3 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 | « webkit/fileapi/local_file_util.cc ('k') | webkit/fileapi/native_file_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/fileapi/local_file_util_unittest.cc
diff --git a/webkit/fileapi/local_file_system_file_util_unittest.cc b/webkit/fileapi/local_file_util_unittest.cc
similarity index 89%
rename from webkit/fileapi/local_file_system_file_util_unittest.cc
rename to webkit/fileapi/local_file_util_unittest.cc
index 5f3906ede56bdaba8baabb8ecb6f5f355d5db9f6..119d919ed35b996cab9eea3917142c5e52b99820 100644
--- a/webkit/fileapi/local_file_system_file_util_unittest.cc
+++ b/webkit/fileapi/local_file_util_unittest.cc
@@ -18,16 +18,16 @@
#include "webkit/fileapi/file_system_path_manager.h"
#include "webkit/fileapi/file_system_test_helper.h"
#include "webkit/fileapi/file_system_types.h"
-#include "webkit/fileapi/local_file_system_file_util.h"
+#include "webkit/fileapi/local_file_util.h"
+#include "webkit/fileapi/native_file_util.h"
namespace fileapi {
-// TODO(dmikurube): Cover all public methods in LocalFileSystemFileUtil.
-class LocalFileSystemFileUtilTest : public testing::Test {
+// TODO(dmikurube): Cover all public methods in LocalFileUtil.
+class LocalFileUtilTest : public testing::Test {
public:
- LocalFileSystemFileUtilTest()
- : local_file_util_(
- new LocalFileSystemFileUtil(new FileSystemFileUtil())),
+ LocalFileUtilTest()
+ : local_file_util_(new LocalFileUtil(new NativeFileUtil())),
callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
}
@@ -46,7 +46,7 @@ class LocalFileSystemFileUtilTest : public testing::Test {
return context;
}
- LocalFileSystemFileUtil* FileUtil() {
+ LocalFileUtil* FileUtil() {
return local_file_util_.get();
}
@@ -94,16 +94,16 @@ class LocalFileSystemFileUtilTest : public testing::Test {
}
private:
- scoped_ptr<LocalFileSystemFileUtil> local_file_util_;
+ scoped_ptr<LocalFileUtil> local_file_util_;
ScopedTempDir data_dir_;
FileSystemTestOriginHelper test_helper_;
- base::ScopedCallbackFactory<LocalFileSystemFileUtilTest> callback_factory_;
+ base::ScopedCallbackFactory<LocalFileUtilTest> callback_factory_;
- DISALLOW_COPY_AND_ASSIGN(LocalFileSystemFileUtilTest);
+ DISALLOW_COPY_AND_ASSIGN(LocalFileUtilTest);
};
-TEST_F(LocalFileSystemFileUtilTest, CreateAndClose) {
+TEST_F(LocalFileUtilTest, CreateAndClose) {
const char *file_name = "test_file";
base::PlatformFile file_handle;
bool created;
@@ -119,7 +119,7 @@ TEST_F(LocalFileSystemFileUtilTest, CreateAndClose) {
FileUtil()->Close(context.get(), file_handle));
}
-TEST_F(LocalFileSystemFileUtilTest, EnsureFileExists) {
+TEST_F(LocalFileUtilTest, EnsureFileExists) {
const char *file_name = "foobar";
bool created;
ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(file_name, &created));
@@ -132,7 +132,7 @@ TEST_F(LocalFileSystemFileUtilTest, EnsureFileExists) {
EXPECT_FALSE(created);
}
-TEST_F(LocalFileSystemFileUtilTest, Truncate) {
+TEST_F(LocalFileUtilTest, Truncate) {
const char *file_name = "truncated";
bool created;
ASSERT_EQ(base::PLATFORM_FILE_OK, EnsureFileExists(file_name, &created));
@@ -148,7 +148,7 @@ TEST_F(LocalFileSystemFileUtilTest, Truncate) {
EXPECT_EQ(1020, GetSize(file_name));
}
-TEST_F(LocalFileSystemFileUtilTest, CopyFile) {
+TEST_F(LocalFileUtilTest, CopyFile) {
const char *from_file = "fromfile";
const char *to_file1 = "tofile1";
const char *to_file2 = "tofile2";
@@ -180,7 +180,7 @@ TEST_F(LocalFileSystemFileUtilTest, CopyFile) {
EXPECT_EQ(1020, GetSize(to_file2));
}
-TEST_F(LocalFileSystemFileUtilTest, CopyDirectory) {
+TEST_F(LocalFileUtilTest, CopyDirectory) {
const char *from_dir = "fromdir";
const char *from_file = "fromdir/fromfile";
const char *to_dir = "todir";
@@ -215,7 +215,7 @@ TEST_F(LocalFileSystemFileUtilTest, CopyDirectory) {
EXPECT_EQ(1020, GetSize(to_file));
}
-TEST_F(LocalFileSystemFileUtilTest, MoveFile) {
+TEST_F(LocalFileUtilTest, MoveFile) {
const char *from_file = "fromfile";
const char *to_file = "tofile";
bool created;
@@ -239,7 +239,7 @@ TEST_F(LocalFileSystemFileUtilTest, MoveFile) {
EXPECT_EQ(1020, GetSize(to_file));
}
-TEST_F(LocalFileSystemFileUtilTest, MoveDirectory) {
+TEST_F(LocalFileUtilTest, MoveDirectory) {
const char *from_dir = "fromdir";
const char *from_file = "fromdir/fromfile";
const char *to_dir = "todir";
« no previous file with comments | « webkit/fileapi/local_file_util.cc ('k') | webkit/fileapi/native_file_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698