Chromium Code Reviews| Index: ui/login/account_picker/md_screen_account_picker.js |
| diff --git a/ui/login/account_picker/md_screen_account_picker.js b/ui/login/account_picker/md_screen_account_picker.js |
| index 88ca6736d29011c8e2b851dff02530fac037c0be..5e4d45c99126421d90a77803d2df92624232a328 100644 |
| --- a/ui/login/account_picker/md_screen_account_picker.js |
| +++ b/ui/login/account_picker/md_screen_account_picker.js |
| @@ -116,6 +116,10 @@ login.createScreen('AccountPickerScreen', 'account-picker', function() { |
| */ |
| onBeforeShow: function(data) { |
| this.showing_ = true; |
| + |
| + this.ownerDocument.addEventListener('click', |
| + this.handleOwnerDocClick_.bind(this)); |
| + |
| chrome.send('loginUIStateChanged', ['account-picker', true]); |
| $('login-header-bar').signinUIState = SIGNIN_UI_STATE.ACCOUNT_PICKER; |
| // Header bar should be always visible on Account Picker screen. |
| @@ -480,11 +484,6 @@ login.createScreen('AccountPickerScreen', 'account-picker', function() { |
| this.lockScreenAppsState_ = state; |
| $('login-header-bar').lockScreenAppsState = state; |
| - // When an lock screen app window is in background - i.e. visible behind |
| - // the lock screen UI - dim the lock screen background, so it's more |
| - // noticeable that the app widow in background is not actionable. |
| - $('background').classList.toggle( |
| - 'dimmed-background', state == LOCK_SCREEN_APPS_STATE.BACKGROUND); |
| }, |
| /** |
| @@ -496,7 +495,8 @@ login.createScreen('AccountPickerScreen', 'account-picker', function() { |
| */ |
| handleOwnerDocClick_: function(event) { |
| if (this.lockScreenAppsState_ != LOCK_SCREEN_APPS_STATE.BACKGROUND || |
|
xiyuan
2017/06/20 21:39:16
nit: strip 1 space before "!=" since you are here.
tbarzic
2017/06/20 22:07:57
Done.
|
| - event.target != $('outer-container')) { |
| + (event.target != $('account-picker') && |
| + event.target != $('version'))) { |
| return; |
| } |
| chrome.send('setLockScreenAppsState', |