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

Unified Diff: ui/file_manager/file_manager/common/js/util.js

Issue 528263002: Hide .crdownload files and prevent users from changing extension to it. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Precompile regular expression. Created 6 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 | « no previous file | ui/file_manager/file_manager/foreground/js/directory_contents.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/file_manager/common/js/util.js
diff --git a/ui/file_manager/file_manager/common/js/util.js b/ui/file_manager/file_manager/common/js/util.js
index 25af53434638b685ac5d95b681ed20b2a6b886a7..757955f6e91fdb278cf7bc7b2fba8297d8eec6ed 100644
--- a/ui/file_manager/file_manager/common/js/util.js
+++ b/ui/file_manager/file_manager/common/js/util.js
@@ -1344,6 +1344,8 @@ util.validateFileName = function(parentEntry, name, filterHiddenOn) {
return Promise.reject(str('ERROR_WHITESPACE_NAME'));
else if (/^(CON|PRN|AUX|NUL|COM[1-9]|LPT[1-9])$/i.test(name))
return Promise.reject(str('ERROR_RESERVED_NAME'));
+ else if (filterHiddenOn && /\.crdownload$/i.test(name))
+ return Promise.reject(str('ERROR_RESERVED_NAME'));
else if (filterHiddenOn && name[0] == '.')
return Promise.reject(str('ERROR_HIDDEN_NAME'));
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/directory_contents.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698