[MD settings] remove selectedSite property

chrome/browser/resources/settings/site_settings/site_list.js

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..8c3894bc0ca7b07cdb33a3fc0d49654208e68779 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,9 @@ Polymer({
   onOriginTap_: function(event) {
     if (!this.enableSiteSettings_)
       return;
-    this.selectedSite = event.model.item;
+    this.selectedSiteForUnitTest = event.model.item;

[tommycli, 2016/12/16 00:39:49]

I think you can also eliminate this by testing for the 'site' key in
URLSearchParams.

https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/route....

Also - I assume you double checked my conjecture that selectedSite is indeed not
needed?

[dschuyler, 2016/12/16 01:11:57]

Done.

     settings.navigateTo(settings.Route.SITE_SETTINGS_SITE_DETAILS,
-        new URLSearchParams('site=' + this.selectedSite.origin));
+        new URLSearchParams('site=' + event.model.item.origin));
   },
 
   /**