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

Unified Diff: chrome/test/data/extensions/api_test/file_system/get_display_path_prettify_mac/test.js

Issue 10823157: Prettify Mac names, rewrite getDisplayPath API call to focus on $HOME (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Actually works on Mac now Created 8 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
Index: chrome/test/data/extensions/api_test/file_system/get_display_path_prettify_mac/test.js
diff --git a/chrome/test/data/extensions/api_test/file_system/get_display_path_prettify/test.js b/chrome/test/data/extensions/api_test/file_system/get_display_path_prettify_mac/test.js
similarity index 65%
copy from chrome/test/data/extensions/api_test/file_system/get_display_path_prettify/test.js
copy to chrome/test/data/extensions/api_test/file_system/get_display_path_prettify_mac/test.js
index 1cf19a154e68debe4c2f29e6f96e0fd173d89421..a0d98cd23e3a8e928762a89ffb9f825a1f5c65d4 100644
--- a/chrome/test/data/extensions/api_test/file_system/get_display_path_prettify/test.js
+++ b/chrome/test/data/extensions/api_test/file_system/get_display_path_prettify_mac/test.js
@@ -7,13 +7,13 @@ chrome.test.runTests([
chrome.fileSystem.chooseFile(chrome.test.callbackPass(function(entry) {
chrome.test.assertEq('gold.txt', entry.name);
- // Test that we can get the display path of the file, and that it begins
- // with the home directory indicator (currently "~").
+ // Test that we can get the display path of the file, and that it does
+ // not begin with a slash. As this test may be run on a wide variety
+ // of machines with different locales, we cannot test the actual prefix.
chrome.fileSystem.getDisplayPath(entry, chrome.test.callbackPass(
function(path) {
- chrome.test.assertTrue(path.indexOf("~/") == 0 || // POSIX
- path.indexOf("~\\") == 0); // Windows
- chrome.test.assertTrue(path.indexOf("gold.txt") >= 0);
+ chrome.test.assertTrue(path.substring(0, 1) != '/');
+ chrome.test.assertTrue(path.indexOf("/gold.txt") >= 0);
}));
}));
}

Powered by Google App Engine
This is Rietveld 408576698