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

Side by Side Diff: chrome/browser/resources/settings/site_settings/site_data.js

Issue 2842303004: MD Settings: update dialogs to focus without ink when using mouse (Closed)
Patch Set: merge 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 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 6 * @fileoverview
7 * 'site-data' handles showing the local storage summary list for all sites. 7 * 'site-data' handles showing the local storage summary list for all sites.
8 */ 8 */
9 9
10 /** 10 /**
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 return loadTimeData.getString('siteSettingsCookieRemoveAllShown'); 102 return loadTimeData.getString('siteSettingsCookieRemoveAllShown');
103 }, 103 },
104 104
105 /** @private */ 105 /** @private */
106 onCloseDialog_: function() { 106 onCloseDialog_: function() {
107 this.$.confirmDeleteDialog.close(); 107 this.$.confirmDeleteDialog.close();
108 }, 108 },
109 109
110 /** @private */ 110 /** @private */
111 onConfirmDeleteDialogClosed_: function() { 111 onConfirmDeleteDialogClosed_: function() {
112 this.$.removeShowingSites.focus(); 112 cr.ui.focusWithoutInk(assert(this.$.removeShowingSites));
113 }, 113 },
114 114
115 /** 115 /**
116 * Shows a dialog to confirm the deletion of multiple sites. 116 * Shows a dialog to confirm the deletion of multiple sites.
117 * @param {!Event} e 117 * @param {!Event} e
118 * @private 118 * @private
119 */ 119 */
120 onRemoveShowingSitesTap_: function(e) { 120 onRemoveShowingSitesTap_: function(e) {
121 e.preventDefault(); 121 e.preventDefault();
122 this.confirmationDeleteMsg_ = loadTimeData.getString( 122 this.confirmationDeleteMsg_ = loadTimeData.getString(
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
156 156
157 /** 157 /**
158 * @param {!{model: !{item: CookieDataSummaryItem}}} event 158 * @param {!{model: !{item: CookieDataSummaryItem}}} event
159 * @private 159 * @private
160 */ 160 */
161 onSiteTap_: function(event) { 161 onSiteTap_: function(event) {
162 settings.navigateTo(settings.Route.SITE_SETTINGS_DATA_DETAILS, 162 settings.navigateTo(settings.Route.SITE_SETTINGS_DATA_DETAILS,
163 new URLSearchParams('site=' + event.model.item.site)); 163 new URLSearchParams('site=' + event.model.item.site));
164 }, 164 },
165 }); 165 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698