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

Unified Diff: chrome/browser/resources/md_downloads/toolbar.js

Issue 1613233003: MD Downloads: make Ctrl+f invoke in-page search (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dl-vulcanize
Patch Set: rework, revulc Created 4 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/resources/md_downloads/toolbar.js
diff --git a/chrome/browser/resources/md_downloads/toolbar.js b/chrome/browser/resources/md_downloads/toolbar.js
index ff9173ee452a3612eb349db0c76cedc94147a62a..c23364b3a9fcb446d468ea8711be89a81473d5ce 100644
--- a/chrome/browser/resources/md_downloads/toolbar.js
+++ b/chrome/browser/resources/md_downloads/toolbar.js
@@ -20,7 +20,7 @@ cr.define('downloads', function() {
reflectToAttribute: true,
type: Boolean,
value: false,
- observer: 'onDownloadsShowingChange_',
+ observer: 'downloadsShowingChanged_',
},
overflowAlign_: {
@@ -39,6 +39,10 @@ cr.define('downloads', function() {
return !this.$['search-input'].getValue() && this.downloadsShowing;
},
+ onFindCommand: function() {
+ this.$['search-input'].showAndFocus();
+ },
+
/** @private */
onClearAllTap_: function() {
assert(this.canClearAll());
@@ -46,7 +50,7 @@ cr.define('downloads', function() {
},
/** @private */
- onDownloadsShowingChange_: function() {
+ downloadsShowingChanged_: function() {
this.updateClearAll_();
},

Powered by Google App Engine
This is Rietveld 408576698