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

Unified Diff: chrome/browser/resources/settings/search_engines_page/search_engine_entry.js

Issue 1713283002: MD Settings: Manage search engines, polish to match latest mocks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@extract_listener_behavior
Patch Set: again Created 4 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/settings/search_engines_page/search_engine_entry.js
diff --git a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.js b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.js
index 9e840ab83224638bcb409f146c2121a0886e097f..373b683a6bf2563669c4a63c9123dc0239d83011 100644
--- a/chrome/browser/resources/settings/search_engines_page/search_engine_entry.js
+++ b/chrome/browser/resources/settings/search_engines_page/search_engine_entry.js
@@ -18,15 +18,31 @@ Polymer({
/** @private {boolean} */
showEditSearchEngineDialog_: Boolean,
+
+ /** @type {boolean} */
+ isDefault: {
+ reflectToAttribute: true,
+ type: Boolean,
+ computed: 'isDefault_(engine)'
+ },
},
/** @private {!settings.SearchEnginesBrowserProxy} */
browserProxy_: null,
+ /** @override */
created: function() {
this.browserProxy_ = settings.SearchEnginesBrowserProxyImpl.getInstance();
},
+ /**
+ * @return {boolean}
+ * @private
+ */
+ isDefault_: function() {
Dan Beam 2016/02/23 02:45:12 nit: computeIsDefault_
dpapad 2016/02/23 21:27:04 Done.
+ return this.engine.default;
+ },
+
/** @private */
onDeleteTap_: function() {
this.browserProxy_.removeSearchEngine(this.engine.modelIndex);

Powered by Google App Engine
This is Rietveld 408576698