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

Side by Side Diff: ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js

Issue 2152753003: MD History: Fix forward slash key not working in search field (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // TODO(tsergeant): Add tests for cr-toolbar-search-field. 5 // TODO(tsergeant): Add tests for cr-toolbar-search-field.
6 Polymer({ 6 Polymer({
7 is: 'cr-toolbar-search-field', 7 is: 'cr-toolbar-search-field',
8 8
9 behaviors: [CrSearchFieldBehavior], 9 behaviors: [CrSearchFieldBehavior],
10 10
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 this.showingSearch = true; 83 this.showingSearch = true;
84 }, 84 },
85 85
86 /** 86 /**
87 * @param {Event} e 87 * @param {Event} e
88 * @private 88 * @private
89 */ 89 */
90 hideSearch_: function(e) { 90 hideSearch_: function(e) {
91 this.showingSearch = false; 91 this.showingSearch = false;
92 e.stopPropagation(); 92 e.stopPropagation();
93 },
94
95 /**
96 * Return if the search field is showing or not.
97 * @return {boolean}
98 */
99 showingSearchStatus: function() {
Dan Beam 2016/07/15 03:07:41 why is this needed if this.showingSearch is alread
lshang 2016/07/15 05:02:09 Done. Deleted.
100 return this.showingSearch;
93 } 101 }
94 }); 102 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698