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

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

Issue 2323693002: Site Settings Desktop: Implement Zoom Levels category. (Closed)
Patch Set: Fix test Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js b/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
index d3b9da62e02d4157532f5ae6b4d1e9696ffb687a..163a093dc93081e7baaf30065f2a052a068cbdc2 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_prefs_browser_proxy.js
@@ -66,6 +66,14 @@ var UsbDeviceDetails;
*/
var UsbDeviceEntry;
+/**
+ * @typedef {{origin: string,
+ * setting: string,
+ * source: string,
+ * zoom: string}}
+ */
+var ZoomLevelEntry;
+
cr.define('settings', function() {
/** @interface */
function SiteSettingsPrefsBrowserProxy() {}
@@ -216,6 +224,18 @@ cr.define('settings', function() {
* profile exists). Returns the results via onIncognitoStatusChanged.
*/
updateIncognitoStatus: function() {},
+
+ /**
+ * Fetches the currently defined zoom levels for sites. Returns the results
+ * via onZoomLevelsChanged.
+ */
+ fetchZoomLevels: function() {},
+
+ /**
+ * Removes a zoom levels for a given host.
+ * @param {string} host The host to remove zoom levels for.
+ */
+ removeZoomLevel: function(host) {},
};
/**
@@ -326,6 +346,16 @@ cr.define('settings', function() {
updateIncognitoStatus: function() {
chrome.send('updateIncognitoStatus');
},
+
+ /** @override */
+ fetchZoomLevels: function() {
+ chrome.send('fetchZoomLevels');
+ },
+
+ /** @override */
+ removeZoomLevel: function(host) {
+ chrome.send('removeZoomLevel', [host]);
+ },
};
return {

Powered by Google App Engine
This is Rietveld 408576698