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

Unified Diff: chrome/browser/resources/options2/browser_options.js

Issue 9405035: Implement ephemeral users (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Comments addressed. Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options2/browser_options.js
diff --git a/chrome/browser/resources/options2/browser_options.js b/chrome/browser/resources/options2/browser_options.js
index ab34db5496e378d44e1e84b13310a47ff9bae49b..a1c320c295fe6b71ebf001fe604009d6dedda3fa 100644
--- a/chrome/browser/resources/options2/browser_options.js
+++ b/chrome/browser/resources/options2/browser_options.js
@@ -207,16 +207,21 @@ cr.define('options', function() {
this.updateAccountPicture_();
- if (cr.commandLine && cr.commandLine.options['--bwsi']) {
- // Disable the screen lock checkbox in guest mode.
- $('enable-screen-lock').disabled = true;
-
- // Hide the startup section in Guest mode.
- $('startup-section').hidden = true;
- } else {
+ if (localStrings.getString('current_user_is_ephemeral') != 'true') {
+ // Enable picture changing for non-ephemeral users.
$('account-picture-wrapper').onclick = function(event) {
OptionsPage.navigateToPage('changePicture');
};
+ } else {
+ // Hide picture changing option for ephemeral users.
Nikita (slow) 2012/03/07 10:03:04 Please remove this code. Already taken care for Gu
use bartfab instead 2012/03/07 11:10:08 Done.
+ $('change-picture-caption').hidden = true;
+
+ if (cr.commandLine && cr.commandLine.options['--bwsi']) {
+ // Disable the screen lock checkbox in guest mode.
+ $('enable-screen-lock').disabled = true;
+
+ // Hide the startup section in guest mode.
+ $('startup-section').hidden = true;
}
$('manage-accounts-button').onclick = function(event) {

Powered by Google App Engine
This is Rietveld 408576698