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

Unified Diff: chrome/browser/chromeos/file_manager/url_util_unittest.cc

Issue 144783002: Simplify directory initialization in Files app. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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/file_manager/url_util_unittest.cc
diff --git a/chrome/browser/chromeos/file_manager/url_util_unittest.cc b/chrome/browser/chromeos/file_manager/url_util_unittest.cc
index 402d210da611326465aa428231e421414ff87bb8..3e0f75b31fe1874e239e6896ca83df62709e6d8e 100644
--- a/chrome/browser/chromeos/file_manager/url_util_unittest.cc
+++ b/chrome/browser/chromeos/file_manager/url_util_unittest.cc
@@ -56,8 +56,10 @@ TEST(FileManagerUrlUtilTest, GetFileManagerMainPageUrlWithParams_NoFileTypes) {
// The escaped query is hard to read. Pretty print the escaped JSON.
EXPECT_EQ("{\n"
" \"defaultExtension\": \"txt\",\n"
- " \"defaultPath\": \"foo.txt\",\n"
+ " \"currentDirectoryPath\": \"/Downloads\",\n"
+ " \"selectionPath\": \"/Downloads/foo.txt\",\n"
" \"shouldReturnLocalPath\": true,\n"
+ " \"targetName\": \"foo.txt\",\n"
" \"title\": \"some title\",\n"
" \"type\": \"open-file\"\n"
"}\n",
@@ -85,7 +87,7 @@ TEST(FileManagerUrlUtilTest,
const GURL url = GetFileManagerMainPageUrlWithParams(
ui::SelectFileDialog::SELECT_OPEN_FILE,
base::UTF8ToUTF16("some title"),
- base::FilePath::FromUTF8Unsafe("foo.txt"),
+ base::FilePath::FromUTF8Unsafe("/Downloads/foo.txt"),
&file_types,
1, // The file type index is 1-based.
FILE_PATH_LITERAL("txt"));
@@ -98,9 +100,11 @@ TEST(FileManagerUrlUtilTest,
// The escaped query is hard to read. Pretty print the escaped JSON.
EXPECT_EQ("{\n"
" \"defaultExtension\": \"txt\",\n"
- " \"defaultPath\": \"foo.txt\",\n"
+ " \"currentDirectoryPath\": \"/Downloads\",\n"
+ " \"selectionPath\": \"/Downloads/foo.txt\",\n"
" \"includeAllFiles\": false,\n"
" \"shouldReturnLocalPath\": false,\n"
+ " \"targetName\": \"foo.txt\",\n"
" \"title\": \"some title\",\n"
" \"type\": \"open-file\",\n"
" \"typeList\": [ {\n"

Powered by Google App Engine
This is Rietveld 408576698