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

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

Issue 2889803002: md settings: Give focus to password prompt on tap. (Closed)
Patch Set: Created 3 years, 7 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 6 * @fileoverview
7 * 'settings-people-page' is the settings page containing sign-in settings. 7 * 'settings-people-page' is the settings page containing sign-in settings.
8 */ 8 */
9 Polymer({ 9 Polymer({
10 is: 'settings-people-page', 10 is: 'settings-people-page',
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 settings.navigateTo(settings.Route.ABOUT); 319 settings.navigateTo(settings.Route.ABOUT);
320 break; 320 break;
321 case settings.StatusAction.ENTER_PASSPHRASE: 321 case settings.StatusAction.ENTER_PASSPHRASE:
322 case settings.StatusAction.NO_ACTION: 322 case settings.StatusAction.NO_ACTION:
323 default: 323 default:
324 settings.navigateTo(settings.Route.SYNC); 324 settings.navigateTo(settings.Route.SYNC);
325 } 325 }
326 }, 326 },
327 327
328 // <if expr="chromeos"> 328 // <if expr="chromeos">
329 /** @private */ 329 /**
330 onConfigureLockTap_: function() { 330 * @param {!Event} e
331 * @private
332 */
333 onConfigureLockTap_: function(e) {
334 // Navigating to the lock screen will always open the password prompt
335 // dialog, so prevent the end of the tap event to focus what is underneath
336 // it, which takes focus from the dialog.
337 e.preventDefault();
331 settings.navigateTo(settings.Route.LOCK_SCREEN); 338 settings.navigateTo(settings.Route.LOCK_SCREEN);
332 }, 339 },
333 // </if> 340 // </if>
334 341
335 /** @private */ 342 /** @private */
336 onManageOtherPeople_: function() { 343 onManageOtherPeople_: function() {
337 // <if expr="not chromeos"> 344 // <if expr="not chromeos">
338 this.syncBrowserProxy_.manageOtherPeople(); 345 this.syncBrowserProxy_.manageOtherPeople();
339 // </if> 346 // </if>
340 // <if expr="chromeos"> 347 // <if expr="chromeos">
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
445 452
446 /** 453 /**
447 * @param {!settings.SyncStatus} syncStatus 454 * @param {!settings.SyncStatus} syncStatus
448 * @return {boolean} Whether to show the "Sign in to Chrome" button. 455 * @return {boolean} Whether to show the "Sign in to Chrome" button.
449 * @private 456 * @private
450 */ 457 */
451 showSignin_: function(syncStatus) { 458 showSignin_: function(syncStatus) {
452 return !!syncStatus.signinAllowed && !syncStatus.signedIn; 459 return !!syncStatus.signinAllowed && !syncStatus.signedIn;
453 }, 460 },
454 }); 461 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698