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

Unified Diff: chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js

Issue 2617823002: GRIT: put <if> and <include> behind comments in .js files to make syntactically valid JS (Closed)
Patch Set: add dep Created 3 years, 11 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/privacy_page/privacy_page_browser_proxy.js
diff --git a/chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js b/chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js
index a630f7423b1c40bb60333a3e284afd03f07d10e5..2c2c4729b7bdc34378ac378a1a02ab6fb600a340 100644
--- a/chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js
+++ b/chrome/browser/resources/settings/privacy_page/privacy_page_browser_proxy.js
@@ -12,18 +12,18 @@ cr.define('settings', function() {
function PrivacyPageBrowserProxy() {}
PrivacyPageBrowserProxy.prototype = {
-<if expr="_google_chrome and not chromeos">
+// <if expr="_google_chrome and not chromeos">
/** @return {!Promise<!MetricsReporting>} */
getMetricsReporting: assertNotReached,
/** @param {boolean} enabled */
setMetricsReportingEnabled: assertNotReached,
-</if>
+// </if>
-<if expr="is_win or is_macosx">
+// <if expr="is_win or is_macosx">
/** Invokes the native certificate manager (used by win and mac). */
showManageSSLCertificates: function() {},
-</if>
+// </if>
/** @return {!Promise<boolean>} */
getSafeBrowsingExtendedReporting: assertNotReached,
@@ -40,7 +40,7 @@ cr.define('settings', function() {
cr.addSingletonGetter(PrivacyPageBrowserProxyImpl);
PrivacyPageBrowserProxyImpl.prototype = {
-<if expr="_google_chrome and not chromeos">
+// <if expr="_google_chrome and not chromeos">
/** @override */
getMetricsReporting: function() {
return cr.sendWithPromise('getMetricsReporting');
@@ -50,7 +50,7 @@ cr.define('settings', function() {
setMetricsReportingEnabled: function(enabled) {
chrome.send('setMetricsReportingEnabled', [enabled]);
},
-</if>
+// </if>
/** @override */
getSafeBrowsingExtendedReporting: function() {
@@ -62,12 +62,12 @@ cr.define('settings', function() {
chrome.send('setSafeBrowsingExtendedReportingEnabled', [enabled]);
},
-<if expr="is_win or is_macosx">
+// <if expr="is_win or is_macosx">
/** @override */
showManageSSLCertificates: function() {
chrome.send('showManageSSLCertificates');
},
-</if>
+// </if>
};
return {

Powered by Google App Engine
This is Rietveld 408576698