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

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

Issue 2793463002: MD Settings: Restore focus after closing various dialogs. (Closed)
Patch Set: more Created 3 years, 8 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-reset-page' is the settings page containing reset 7 * 'settings-reset-page' is the settings page containing reset
8 * settings. 8 * settings.
9 * 9 *
10 * Example: 10 * Example:
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 52
53 /** @private */ 53 /** @private */
54 onShowResetProfileDialog_: function() { 54 onShowResetProfileDialog_: function() {
55 settings.navigateTo(settings.Route.RESET_DIALOG, 55 settings.navigateTo(settings.Route.RESET_DIALOG,
56 new URLSearchParams('origin=userclick')); 56 new URLSearchParams('origin=userclick'));
57 }, 57 },
58 58
59 /** @private */ 59 /** @private */
60 onResetProfileDialogClose_: function() { 60 onResetProfileDialogClose_: function() {
61 settings.navigateToPreviousRoute(); 61 settings.navigateToPreviousRoute();
62 this.$$('#resetProfile button[is=paper-icon-button-light').focus();
tommycli 2017/04/03 15:39:50 Are we very confident this won't be null? Also I t
dpapad 2017/04/03 19:23:05 Fixed closing bracket. Regarding whether it can be
tommycli 2017/04/03 19:30:51 Acknowledged. I would probably still recommend giv
dpapad 2017/04/04 00:03:16 Done.
62 }, 63 },
63 64
64 // <if expr="chromeos"> 65 // <if expr="chromeos">
65 /** 66 /**
66 * @param {!Event} e 67 * @param {!Event} e
67 * @private 68 * @private
68 */ 69 */
69 onShowPowerwashDialog_: function(e) { 70 onShowPowerwashDialog_: function(e) {
70 e.preventDefault(); 71 e.preventDefault();
71 this.showPowerwashDialog_ = true; 72 this.showPowerwashDialog_ = true;
72 }, 73 },
73 74
74 /** @private */ 75 /** @private */
75 onPowerwashDialogClose_: function() { 76 onPowerwashDialogClose_: function() {
76 this.showPowerwashDialog_ = false; 77 this.showPowerwashDialog_ = false;
78 this.$$('#powerwash button[is=paper-icon-button-light').focus();
77 }, 79 },
78 // </if> 80 // </if>
79 }); 81 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698