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

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

Issue 1698093007: Convert SiteSettingsCategory to use the HostContentSettingsMap instead of (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
(Empty)
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
3 // found in the LICENSE file.
4
5 (function() {
6 'use strict';
7
8 Polymer({
9 is: 'prefs-private-api',
10
11 /**
12 * Sets the default value for a site settings category.
13 * @param {number} contentType The category to change.
14 * @param {number} defaultValue The value to set as default.
15 */
16 setDefaultValueForContentType: function(contentType, defaultValue) {
17 chrome.send('setDefaultValueForContentType', [contentType, defaultValue]);
18 },
19
20 /**
21 * Gets the default value for a site settings category.
22 * @param {number} contentType The category to change.
23 * @return {Promise}
24 */
25 getDefaultValueForContentType: function(contentType) {
26 return cr.sendWithPromise('getDefaultValueForContentType', contentType);
27 },
28 });
29 })();
Finnur 2016/02/16 15:41:04 At first, I had a corresponding cr singleton objec
tommycli 2016/02/17 18:18:06 After a long discussion with dpapad, we agreed tha
Finnur 2016/02/18 20:38:17 Done. I'm fine with that, as long as this doesn't
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698