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

Unified Diff: chrome/browser/resources/settings/site_settings/site_list.js

Issue 2580653003: [MD settings] remove selectedSite property (Closed)
Patch Set: nits Created 4 years 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/site_settings/site_list.js
diff --git a/chrome/browser/resources/settings/site_settings/site_list.js b/chrome/browser/resources/settings/site_settings/site_list.js
index 188cb67c13b40abb04953f160684c302ade108d2..9564724efad82dcacfda949e6d5c3fc29b7e5182 100644
--- a/chrome/browser/resources/settings/site_settings/site_list.js
+++ b/chrome/browser/resources/settings/site_settings/site_list.js
@@ -35,15 +35,6 @@ Polymer({
},
/**
- * The site that was selected by the user in the dropdown list.
- * @type {SiteException}
- */
- selectedSite: {
- type: Object,
- notify: true,
- },
-
- /**
* The site serving as the model for the currently open action menu.
* @private {?SiteException}
*/
@@ -404,9 +395,8 @@ Polymer({
onOriginTap_: function(event) {
if (!this.enableSiteSettings_)
return;
- this.selectedSite = event.model.item;
settings.navigateTo(settings.Route.SITE_SETTINGS_SITE_DETAILS,
- new URLSearchParams('site=' + this.selectedSite.origin));
+ new URLSearchParams('site=' + event.model.item.origin));
},
/**

Powered by Google App Engine
This is Rietveld 408576698