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

Side by Side Diff: chrome/browser/resources/settings/site_settings/site_settings_behavior.js

Issue 1982113002: MD Settings: Fix names of icons which had been moved to shared iconset (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pdf_icons
Patch Set: Created 4 years, 7 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
« no previous file with comments | « chrome/browser/resources/settings/site_settings/site_settings_behavior.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 Behavior common to Site Settings classes. 6 * @fileoverview Behavior common to Site Settings classes.
7 */ 7 */
8 8
9 /** @polymerBehavior */ 9 /** @polymerBehavior */
10 var SiteSettingsBehaviorImpl = { 10 var SiteSettingsBehaviorImpl = {
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 * @return {string} The id of the icon for the given category. 96 * @return {string} The id of the icon for the given category.
97 * @protected 97 * @protected
98 */ 98 */
99 computeIconForContentCategory: function(category) { 99 computeIconForContentCategory: function(category) {
100 switch (category) { 100 switch (category) {
101 case settings.ContentSettingsTypes.CAMERA: 101 case settings.ContentSettingsTypes.CAMERA:
102 return 'settings:videocam'; 102 return 'settings:videocam';
103 case settings.ContentSettingsTypes.COOKIES: 103 case settings.ContentSettingsTypes.COOKIES:
104 return 'settings:cookie'; 104 return 'settings:cookie';
105 case settings.ContentSettingsTypes.FULLSCREEN: 105 case settings.ContentSettingsTypes.FULLSCREEN:
106 return 'settings:fullscreen'; 106 return 'cr:fullscreen';
107 case settings.ContentSettingsTypes.GEOLOCATION: 107 case settings.ContentSettingsTypes.GEOLOCATION:
108 return 'settings:location-on'; 108 return 'settings:location-on';
109 case settings.ContentSettingsTypes.IMAGES: 109 case settings.ContentSettingsTypes.IMAGES:
110 return 'settings:photo'; 110 return 'settings:photo';
111 case settings.ContentSettingsTypes.JAVASCRIPT: 111 case settings.ContentSettingsTypes.JAVASCRIPT:
112 return 'settings:input'; 112 return 'settings:input';
113 case settings.ContentSettingsTypes.MIC: 113 case settings.ContentSettingsTypes.MIC:
114 return 'settings:mic'; 114 return 'settings:mic';
115 case settings.ContentSettingsTypes.NOTIFICATIONS: 115 case settings.ContentSettingsTypes.NOTIFICATIONS:
116 return 'settings:notifications'; 116 return 'settings:notifications';
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 * @private 258 * @private
259 */ 259 */
260 computeSiteIcon: function(site) { 260 computeSiteIcon: function(site) {
261 var url = this.ensureUrlHasScheme(site.originForDisplay); 261 var url = this.ensureUrlHasScheme(site.originForDisplay);
262 return 'background-image: ' + cr.icon.getFaviconImageSet(url); 262 return 'background-image: ' + cr.icon.getFaviconImageSet(url);
263 }, 263 },
264 }; 264 };
265 265
266 /** @polymerBehavior */ 266 /** @polymerBehavior */
267 var SiteSettingsBehavior = [SiteSettingsBehaviorImpl]; 267 var SiteSettingsBehavior = [SiteSettingsBehaviorImpl];
OLDNEW
« no previous file with comments | « chrome/browser/resources/settings/site_settings/site_settings_behavior.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698