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

Unified Diff: chrome/browser/resources/file_manager/foreground/js/file_manager.js

Issue 216513004: Files.app: Add a test to create new folder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed. Created 6 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/resources/file_manager/foreground/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/foreground/js/file_manager.js b/chrome/browser/resources/file_manager/foreground/js/file_manager.js
index aa3bd7250950f1b188fd38113fbc13b5eaafa945..8baa8828111ad8fd8d6dc8c4c0b416779a8cf625 100644
--- a/chrome/browser/resources/file_manager/foreground/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/foreground/js/file_manager.js
@@ -2462,13 +2462,13 @@ var BOTTOM_MARGIN_FOR_PREVIEW_PANEL_PX = 52;
event.stopPropagation();
}
- switch (util.getKeyModifiers(event) + event.keyCode) {
- case '27': // Escape
+ switch (util.getKeyModifiers(event) + event.keyIdentifier) {
+ case 'U+001B': // Escape
this.cancelRename_();
event.preventDefault();
break;
- case '13': // Enter
+ case 'Enter':
this.commitRename_();
event.preventDefault();
break;

Powered by Google App Engine
This is Rietveld 408576698