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

Side by Side Diff: chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js

Issue 2954863003: MD Settings: Convert all browser proxies to use ES6 class syntax. (Closed)
Patch Set: Remove @constructor annotations. Created 3 years, 5 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 /** @fileoverview Handles interprocess communcation for the privacy page. */ 5 /** @fileoverview Handles interprocess communcation for the privacy page. */
6 6
7 /** @typedef {{enabled: boolean, managed: boolean}} */ 7 /** @typedef {{enabled: boolean, managed: boolean}} */
8 var MetricsReporting; 8 var MetricsReporting;
9 9
10 cr.define('settings', function() { 10 cr.define('settings', function() {
11 /** @interface */ 11 /** @interface */
12 function PrivacyPageBrowserProxy() {} 12 class PrivacyPageBrowserProxy {
13
14 PrivacyPageBrowserProxy.prototype = {
15 // <if expr="_google_chrome and not chromeos"> 13 // <if expr="_google_chrome and not chromeos">
16 /** @return {!Promise<!MetricsReporting>} */ 14 /** @return {!Promise<!MetricsReporting>} */
17 getMetricsReporting: assertNotReached, 15 getMetricsReporting() {}
18 16
19 /** @param {boolean} enabled */ 17 /** @param {boolean} enabled */
20 setMetricsReportingEnabled: assertNotReached, 18 setMetricsReportingEnabled(enabled) {}
19
21 // </if> 20 // </if>
22 21
23 // <if expr="is_win or is_macosx"> 22 // <if expr="is_win or is_macosx">
24 /** Invokes the native certificate manager (used by win and mac). */ 23 /** Invokes the native certificate manager (used by win and mac). */
25 showManageSSLCertificates: function() {}, 24 showManageSSLCertificates() {}
25
26 // </if> 26 // </if>
27 27
28 /** @return {!Promise<boolean>} */ 28 /** @return {!Promise<boolean>} */
29 getSafeBrowsingExtendedReporting: assertNotReached, 29 getSafeBrowsingExtendedReporting() {}
30 30
31 /** @param {boolean} enabled */ 31 /** @param {boolean} enabled */
32 setSafeBrowsingExtendedReportingEnabled: assertNotReached, 32 setSafeBrowsingExtendedReportingEnabled(enabled) {}
33 }; 33 }
34 34
35 /** 35 /**
36 * @constructor
37 * @implements {settings.PrivacyPageBrowserProxy} 36 * @implements {settings.PrivacyPageBrowserProxy}
38 */ 37 */
39 function PrivacyPageBrowserProxyImpl() {} 38 class PrivacyPageBrowserProxyImpl {
40 cr.addSingletonGetter(PrivacyPageBrowserProxyImpl);
41
42 PrivacyPageBrowserProxyImpl.prototype = {
43 // <if expr="_google_chrome and not chromeos"> 39 // <if expr="_google_chrome and not chromeos">
44 /** @override */ 40 /** @override */
45 getMetricsReporting: function() { 41 getMetricsReporting() {
46 return cr.sendWithPromise('getMetricsReporting'); 42 return cr.sendWithPromise('getMetricsReporting');
47 }, 43 }
48 44
49 /** @override */ 45 /** @override */
50 setMetricsReportingEnabled: function(enabled) { 46 setMetricsReportingEnabled(enabled) {
51 chrome.send('setMetricsReportingEnabled', [enabled]); 47 chrome.send('setMetricsReportingEnabled', [enabled]);
52 }, 48 }
49
53 // </if> 50 // </if>
54 51
55 /** @override */ 52 /** @override */
56 getSafeBrowsingExtendedReporting: function() { 53 getSafeBrowsingExtendedReporting() {
57 return cr.sendWithPromise('getSafeBrowsingExtendedReporting'); 54 return cr.sendWithPromise('getSafeBrowsingExtendedReporting');
58 }, 55 }
59 56
60 /** @override */ 57 /** @override */
61 setSafeBrowsingExtendedReportingEnabled: function(enabled) { 58 setSafeBrowsingExtendedReportingEnabled(enabled) {
62 chrome.send('setSafeBrowsingExtendedReportingEnabled', [enabled]); 59 chrome.send('setSafeBrowsingExtendedReportingEnabled', [enabled]);
63 }, 60 }
64 61
65 // <if expr="is_win or is_macosx"> 62 // <if expr="is_win or is_macosx">
66 /** @override */ 63 /** @override */
67 showManageSSLCertificates: function() { 64 showManageSSLCertificates() {
68 chrome.send('showManageSSLCertificates'); 65 chrome.send('showManageSSLCertificates');
69 }, 66 }
70 // </if> 67 // </if>
71 }; 68 }
69
70 cr.addSingletonGetter(PrivacyPageBrowserProxyImpl);
72 71
73 return { 72 return {
74 PrivacyPageBrowserProxy: PrivacyPageBrowserProxy, 73 PrivacyPageBrowserProxy: PrivacyPageBrowserProxy,
75 PrivacyPageBrowserProxyImpl: PrivacyPageBrowserProxyImpl, 74 PrivacyPageBrowserProxyImpl: PrivacyPageBrowserProxyImpl,
76 }; 75 };
77 }); 76 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698