Display content settings for extensions with extension name.

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

Index: chrome/browser/resources/settings/site_settings/site_list.html
diff --git a/chrome/browser/resources/settings/site_settings/site_list.html b/chrome/browser/resources/settings/site_settings/site_list.html
index d3e83dc04b0bea8e54c5d757c75539a3d863df8f..c2be8e3e462f9750a1440387fa1853d3c3b0bdd5 100644
--- a/chrome/browser/resources/settings/site_settings/site_list.html
+++ b/chrome/browser/resources/settings/site_settings/site_list.html
@@ -66,7 +66,7 @@
             <div class="start layout horizontal center" on-tap="onOriginTap_"
                 actionable$="[[enableSiteSettings_]]">
               <div class="favicon-image"
-                  style$="[[computeSiteIcon(item.originForDisplay)]]">
+                  style$="[[computeSiteIcon(item.origin)]]">

[dschuyler, 2016/11/28 21:58:42]

I'm not clear on why this change was made origin rather
than leaving it as originForDisplay or maybe originForFavicon.
Could ya tell me more about this change?

[dullweber, 2016/11/29 09:34:18]

In case of extensions, the originForDisplay is now a text, e.g. "Bookmark
Manager". The favicon function needs a "chrome-extension://asdfghjkl..." url.
Maybe I should rename "originForDisplay" to something like "name" to indicate
that it might not have anything to do with an origin?
I don't need originForFavicon for the site-list because origin already contains
the scheme. Only Zoomlevels need a separate handling for this.

[dschuyler, 2016/11/29 22:46:59]

I like the variable label of |displayName| used elsewhere in this CL.
That could be an optional or separate CL. I.e. I don't want to hold up
this CL for that change, but I think it would be nicer than |origin|
which may not be an origin. 

Maybe the C++ side could set |origin| and
|displayName| separately rather than 
interpreting it on the JS side.

Said another way, I'm suggesting this series of steps:
- that this change back to originForDisplay
- originForDisplay be renamed as displayName
- displayName set on the C++ side to be either an origin intended for display or
an extension name (or something else in the future). [With this CL setting
displayName - does displayName get set in all cases?]
- origin with then only be an origin (or maybe empty if there is not relevant
origin).

WDYT? I'm open to chatting about this outside of this CL if you prefer.

               </div>
               <div class="middle">
                 <div class="selectable">[[item.originForDisplay]]</div>