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

Side by Side Diff: chrome/browser/resources/settings/search_engines_page/search_engine_entry.js

Issue 1368733002: Introduce cr-settings-input and eliminate cr-input (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue_532540_elim_cr_toggle_button
Patch Set: Rebase Created 5 years, 2 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 /** 5 /**
6 * @fileoverview 'cr-search-engine-entry' is a component for showing a search 6 * @fileoverview 'cr-search-engine-entry' is a component for showing a search
7 * engine with its name, domain and query URL. 7 * engine with its name, domain and query URL.
8 * 8 *
9 * @group Chrome Settings Elements 9 * @group Chrome Settings Elements
10 * @element cr-search-engine-entry 10 * @element cr-search-engine-entry
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 this.$.keywordField.disabled = true; 46 this.$.keywordField.disabled = true;
47 this.$.queryURLField.disabled = true; 47 this.$.queryURLField.disabled = true;
48 48
49 this.$.checkIcon.hidden = true; 49 this.$.checkIcon.hidden = true;
50 this.$.deleteIcon.hidden = true; 50 this.$.deleteIcon.hidden = true;
51 }, 51 },
52 52
53 /** @private */ 53 /** @private */
54 fieldChanged_: function() { 54 fieldChanged_: function() {
55 // NOTE: This currently doesn't fire in response to a change event from the 55 // NOTE: This currently doesn't fire in response to a change event from the
56 // cr-input, even though it should. This Polymer change should fix the 56 // paper-input, even though it should. This Polymer change should fix the
57 // issue: https://github.com/PolymerElements/paper-input/pull/33 57 // issue: https://github.com/PolymerElements/paper-input/pull/33
58 chrome.searchEnginesPrivate.updateSearchEngine( 58 chrome.searchEnginesPrivate.updateSearchEngine(
59 this.engine.guid, 59 this.engine.guid,
60 this.$.domainField.value, 60 this.$.domainField.value,
61 this.$.keywordField.value, 61 this.$.keywordField.value,
62 this.$.queryURLField.value); 62 this.$.queryURLField.value);
63 this.makeNotEditable_(); 63 this.makeNotEditable_();
64 } 64 }
65 }); 65 });
66
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698