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

Side by Side Diff: chrome/browser/resources/settings/people_page/sync_browser_proxy.js

Issue 2102803002: Settings People Revamp: Add special Disconnect logic for domain profiles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make behavior same as Options Created 4 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 /** 5 /**
6 * @fileoverview A helper object used from the the People section to get the 6 * @fileoverview A helper object used from the the People section to get the
7 * status of the sync backend and user preferences on what data to sync. Used 7 * status of the sync backend and user preferences on what data to sync. Used
8 * for both Chrome browser and ChromeOS. 8 * for both Chrome browser and ChromeOS.
9 */ 9 */
10 cr.exportPath('settings'); 10 cr.exportPath('settings');
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 <if expr="not chromeos"> 93 <if expr="not chromeos">
94 /** 94 /**
95 * Starts the signin process for the user. Does nothing if the user is 95 * Starts the signin process for the user. Does nothing if the user is
96 * already signed in. 96 * already signed in.
97 */ 97 */
98 startSignIn: function() {}, 98 startSignIn: function() {},
99 99
100 /** 100 /**
101 * Signs out the signed-in user. 101 * Signs out the signed-in user.
102 * @param {boolean} deleteProfile 102 * @param {boolean} deleteProfile
103 * @param {boolean} domainManagedProfile
dpapad 2016/06/28 22:17:22 It is a bit odd that we send information from JS t
tommycli 2016/06/28 23:26:00 Done. Good point. This does subtly change the beha
103 */ 104 */
104 signOut: function(deleteProfile) {}, 105 signOut: function(deleteProfile, domainManagedProfile) {},
105 106
106 /** 107 /**
107 * Opens the multi-profile user manager. 108 * Opens the multi-profile user manager.
108 */ 109 */
109 manageOtherPeople: function() {}, 110 manageOtherPeople: function() {},
110 </if> 111 </if>
111 112
112 /** 113 /**
113 * Gets the current sync status. 114 * Gets the current sync status.
114 * @return {!Promise<!settings.SyncStatus>} 115 * @return {!Promise<!settings.SyncStatus>}
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 <if expr="not chromeos"> 160 <if expr="not chromeos">
160 /** @override */ 161 /** @override */
161 startSignIn: function() { 162 startSignIn: function() {
162 // TODO(tommycli): Currently this is always false, but this will become 163 // TODO(tommycli): Currently this is always false, but this will become
163 // a parameter once supervised users are implemented in MD Settings. 164 // a parameter once supervised users are implemented in MD Settings.
164 var creatingSupervisedUser = false; 165 var creatingSupervisedUser = false;
165 chrome.send('SyncSetupStartSignIn', [creatingSupervisedUser]); 166 chrome.send('SyncSetupStartSignIn', [creatingSupervisedUser]);
166 }, 167 },
167 168
168 /** @override */ 169 /** @override */
169 signOut: function(deleteProfile) { 170 signOut: function(deleteProfile, domainManagedProfile) {
170 chrome.send('SyncSetupStopSyncing', [deleteProfile]); 171 chrome.send('SyncSetupStopSyncing',
172 [deleteProfile, domainManagedProfile]);
171 }, 173 },
172 174
173 /** @override */ 175 /** @override */
174 manageOtherPeople: function() { 176 manageOtherPeople: function() {
175 chrome.send('SyncSetupManageOtherPeople'); 177 chrome.send('SyncSetupManageOtherPeople');
176 }, 178 },
177 </if> 179 </if>
178 180
179 /** @override */ 181 /** @override */
180 getSyncStatus: function() { 182 getSyncStatus: function() {
(...skipping 27 matching lines...) Expand all
208 chrome.metricsPrivate.recordUserAction( 210 chrome.metricsPrivate.recordUserAction(
209 'Signin_AccountSettings_GoogleActivityControlsClicked'); 211 'Signin_AccountSettings_GoogleActivityControlsClicked');
210 window.open(loadTimeData.getString('activityControlsUrl')); 212 window.open(loadTimeData.getString('activityControlsUrl'));
211 } 213 }
212 }; 214 };
213 215
214 return { 216 return {
215 SyncBrowserProxyImpl: SyncBrowserProxyImpl, 217 SyncBrowserProxyImpl: SyncBrowserProxyImpl,
216 }; 218 };
217 }); 219 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698