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

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

Issue 12381076: Files.app: Disables browser shortcut keys. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index 4605f4f142a09b095835104958c2a788361d9175..ae8afec4bffdc4ab0a0157d9bfc60df62c9fafad 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -833,6 +833,8 @@ DialogType.isModal = function(type) {
this.fileTypeSelector_ = this.dialogDom_.querySelector('#file-type');
this.initFileTypeFilter_();
+ util.disableBrowserShortcutKeys(this.document_);
+
this.updateWindowState_();
// Populate the static localized strings.
i18nTemplate.process(this.document_, loadTimeData);
@@ -2403,12 +2405,6 @@ DialogType.isModal = function(type) {
event.preventDefault();
return;
- case 'Ctrl-79': // Disable native Ctrl-O (open file).
- case 'Ctrl-83': // Disable native Ctrl-S (save as).
- case 'Ctrl-85': // Disable native Ctrl-U (view source).
- event.preventDefault();
- return;
-
case '27': // Escape => Cancel dialog.
if (this.copyManager_ &&
this.copyManager_.getStatus().totalFiles != 0) {

Powered by Google App Engine
This is Rietveld 408576698