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

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

Issue 2687553003: MD Settings: Fix site data details regression and add test. (Closed)
Patch Set: resetRoutForTesting() Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 A helper object used from the "Site Settings" section to 6 * @fileoverview A helper object used from the "Site Settings" section to
7 * interact with the content settings prefs. 7 * interact with the content settings prefs.
8 */ 8 */
9 9
10 /** 10 /**
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 /** 99 /**
100 * Sets the default value for a site settings category. 100 * Sets the default value for a site settings category.
101 * @param {string} contentType The name of the category to change. 101 * @param {string} contentType The name of the category to change.
102 * @param {string} defaultValue The name of the value to set as default. 102 * @param {string} defaultValue The name of the value to set as default.
103 */ 103 */
104 setDefaultValueForContentType: function(contentType, defaultValue) {}, 104 setDefaultValueForContentType: function(contentType, defaultValue) {},
105 105
106 /** 106 /**
107 * Gets the cookie details for a particular site. 107 * Gets the cookie details for a particular site.
108 * @param {string} site The name of the site. 108 * @param {string} site The name of the site.
109 * @return {!Promise<!CookieDataSummaryItem>} 109 * @return {!Promise<!CookieList>}
110 */ 110 */
111 getCookieDetails: function(site) {}, 111 getCookieDetails: function(site) {},
112 112
113 /** 113 /**
114 * Gets the default value for a site settings category. 114 * Gets the default value for a site settings category.
115 * @param {string} contentType The name of the category to query. 115 * @param {string} contentType The name of the category to query.
116 * @return {!Promise<!DefaultContentSetting>} 116 * @return {!Promise<!DefaultContentSetting>}
117 */ 117 */
118 getDefaultValueForContentType: function(contentType) {}, 118 getDefaultValueForContentType: function(contentType) {},
119 119
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 removeZoomLevel: function(host) { 416 removeZoomLevel: function(host) {
417 chrome.send('removeZoomLevel', [host]); 417 chrome.send('removeZoomLevel', [host]);
418 }, 418 },
419 }; 419 };
420 420
421 return { 421 return {
422 SiteSettingsPrefsBrowserProxy: SiteSettingsPrefsBrowserProxy, 422 SiteSettingsPrefsBrowserProxy: SiteSettingsPrefsBrowserProxy,
423 SiteSettingsPrefsBrowserProxyImpl: SiteSettingsPrefsBrowserProxyImpl, 423 SiteSettingsPrefsBrowserProxyImpl: SiteSettingsPrefsBrowserProxyImpl,
424 }; 424 };
425 }); 425 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698