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

Unified Diff: chrome/browser/resources/settings/people_page/sync_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/people_page/sync_browser_proxy.js
diff --git a/chrome/browser/resources/settings/people_page/sync_browser_proxy.js b/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
index be5c2b6d98523f5b192877738054ea7951f7bd11..1d332be06d7a4515a43a18cbf691e2a2ba50aa15 100644
--- a/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
+++ b/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
@@ -105,7 +105,7 @@ cr.define('settings', function() {
function SyncBrowserProxy() {}
SyncBrowserProxy.prototype = {
-<if expr="not chromeos">
+// <if expr="not chromeos">
/**
* Starts the signin process for the user. Does nothing if the user is
* already signed in.
@@ -122,14 +122,14 @@ cr.define('settings', function() {
* Opens the multi-profile user manager.
*/
manageOtherPeople: function() {},
-</if>
+// </if>
-<if expr="chromeos">
+// <if expr="chromeos">
/**
* Signs the user out.
*/
attemptUserExit: function() {},
-</if>
+// </if>
/**
* Gets the current sync status.
@@ -178,7 +178,7 @@ cr.define('settings', function() {
cr.addSingletonGetter(SyncBrowserProxyImpl);
SyncBrowserProxyImpl.prototype = {
-<if expr="not chromeos">
+// <if expr="not chromeos">
/** @override */
startSignIn: function() {
chrome.send('SyncSetupStartSignIn');
@@ -193,13 +193,13 @@ cr.define('settings', function() {
manageOtherPeople: function() {
chrome.send('SyncSetupManageOtherPeople');
},
-</if>
-<if expr="chromeos">
+// </if>
+// <if expr="chromeos">
/** @override */
attemptUserExit: function() {
return chrome.send('AttemptUserExit');
},
-</if>
+// </if>
/** @override */
getSyncStatus: function() {

Powered by Google App Engine
This is Rietveld 408576698