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

Unified Diff: chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc

Issue 13149003: drive: Use "/drive/root" namespace and fix Files app and tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge glitches. Created 7 years, 9 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
Index: chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc
diff --git a/chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc b/chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc
index 354b0b0b2122474142f58a3af1d4eec5abc1e58a..e011f668a1178cc82f838bd2d5acb7d1cf06a67c 100644
--- a/chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc
+++ b/chrome/browser/chromeos/drive/drive_prefetcher_unittest.cc
@@ -87,34 +87,39 @@ ACTION_P(MockReadDirectory, test_entries) {
callback.Run(DRIVE_FILE_OK, false /* hide_hosted_document */, entries.Pass());
}
+#define FPL FILE_PATH_LITERAL
+
const TestEntry kEmptyDrive[] = {
- { FILE_PATH_LITERAL("drive"), TYPE_DIRECTORY, 0, 0, "id:drive" },
+ { FPL("drive/root"), TYPE_DIRECTORY, 0, 0, "id:drive" },
};
const TestEntry kOneFileDrive[] = {
- { FILE_PATH_LITERAL("drive"), TYPE_DIRECTORY, 0, 0, "id:drive" },
- { FILE_PATH_LITERAL("drive/abc.txt"), TYPE_REGULAR_FILE, 1, 0, "id:abc" },
+ { FPL("drive/root"), TYPE_DIRECTORY, 0, 0, "id:drive" },
+ { FPL("drive/root/abc.txt"), TYPE_REGULAR_FILE, 1, 0,
+ "id:abc" },
};
const char* kExpectedOneFile[] = { "id:abc" };
const TestEntry kComplexDrive[] = {
- // Path Type Access Modify ID
- { FILE_PATH_LITERAL("drive"), TYPE_DIRECTORY, 0, 0, "id:root" },
- { FILE_PATH_LITERAL("drive/a"), TYPE_DIRECTORY, 0, 0, "id:a" },
- { FILE_PATH_LITERAL("drive/a/foo.txt"), TYPE_REGULAR_FILE, 3, 2, "id:foo1" },
- { FILE_PATH_LITERAL("drive/a/b"), TYPE_DIRECTORY, 8, 0, "id:b" },
- { FILE_PATH_LITERAL("drive/a/bar.jpg"), TYPE_REGULAR_FILE, 5, 0, "id:bar1",
- 999 },
- { FILE_PATH_LITERAL("drive/a/b/x.gdoc"), TYPE_HOSTED_FILE, 7, 0, "id:new" },
- { FILE_PATH_LITERAL("drive/a/buz.zip"), TYPE_REGULAR_FILE, 4, 0, "id:buz1" },
- { FILE_PATH_LITERAL("drive/a/old.gdoc"), TYPE_HOSTED_FILE, 1, 0, "id:old" },
- { FILE_PATH_LITERAL("drive/c"), TYPE_DIRECTORY, 0, 0, "id:c" },
- { FILE_PATH_LITERAL("drive/c/foo.txt"), TYPE_REGULAR_FILE, 3, 1, "id:foo2" },
- { FILE_PATH_LITERAL("drive/c/buz.zip"), TYPE_REGULAR_FILE, 1, 0, "id:buz2" },
- { FILE_PATH_LITERAL("drive/bar.jpg"), TYPE_REGULAR_FILE, 6, 0, "id:bar2" },
+ // Path Type Access Modify ID
+ { FPL("drive/root"), TYPE_DIRECTORY, 0, 0, "id:root" },
+ { FPL("drive/root/a"), TYPE_DIRECTORY, 0, 0, "id:a" },
+ { FPL("drive/root/a/foo.txt"), TYPE_REGULAR_FILE, 3, 2, "id:foo1" },
+ { FPL("drive/root/a/b"), TYPE_DIRECTORY, 8, 0, "id:b" },
+ { FPL("drive/root/a/bar.jpg"), TYPE_REGULAR_FILE, 5, 0, "id:bar1",
+ 999 },
+ { FPL("drive/root/a/b/x.gdoc"), TYPE_HOSTED_FILE, 7, 0, "id:new" },
+ { FPL("drive/root/a/buz.zip"), TYPE_REGULAR_FILE, 4, 0, "id:buz1" },
+ { FPL("drive/root/a/old.gdoc"), TYPE_HOSTED_FILE, 1, 0, "id:old" },
+ { FPL("drive/root/c"), TYPE_DIRECTORY, 0, 0, "id:c" },
+ { FPL("drive/root/c/foo.txt"), TYPE_REGULAR_FILE, 3, 1, "id:foo2" },
+ { FPL("drive/root/c/buz.zip"), TYPE_REGULAR_FILE, 1, 0, "id:buz2" },
+ { FPL("drive/root/bar.jpg"), TYPE_REGULAR_FILE, 6, 0, "id:bar2" },
};
+#undef FPL
+
const char* kTop3Files[] = {
"id:bar2", // The file with the largest timestamp
// "bar1" is the second latest, but its file size is over limit.
« no previous file with comments | « chrome/browser/chromeos/drive/drive_file_system_util.h ('k') | chrome/browser/chromeos/drive/drive_resource_metadata.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698