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

Side by Side Diff: chrome/browser/resources/chromeos/keyboard_overlay.js

Issue 2865793002: Remove the BackspaceGoesBack flag. (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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // <include src="keyboard_overlay_data.js"> 5 // <include src="keyboard_overlay_data.js">
6 6
7 var BASE_KEYBOARD = { 7 var BASE_KEYBOARD = {
8 top: 0, 8 top: 0,
9 left: 0, 9 left: 0,
10 width: 1237, 10 width: 1237,
(...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after
220 220
221 if (!isDisplayUIScalingEnabled()) { 221 if (!isDisplayUIScalingEnabled()) {
222 // Zoom screen in 222 // Zoom screen in
223 delete shortcutDataCache['+<>CTRL<>SHIFT']; 223 delete shortcutDataCache['+<>CTRL<>SHIFT'];
224 // Zoom screen out 224 // Zoom screen out
225 delete shortcutDataCache['-<>CTRL<>SHIFT']; 225 delete shortcutDataCache['-<>CTRL<>SHIFT'];
226 // Reset screen zoom 226 // Reset screen zoom
227 delete shortcutDataCache['0<>CTRL<>SHIFT']; 227 delete shortcutDataCache['0<>CTRL<>SHIFT'];
228 } 228 }
229 229
230 if (!loadTimeData.getBoolean('backspaceGoesBackFeatureEnabled')) { 230 delete shortcutDataCache['backspace'];
231 // If the "backspace key goes back" experiment is not enabled, then we 231 delete shortcutDataCache['backspace<>SHIFT'];
Peter Kasting 2017/05/06 04:56:13 Do we need these lines anymore?
ojan 2017/05/10 18:07:06 I don't know. I was just maintaining behavior with
michaelpg 2017/05/10 23:41:51 shortcutDataCache starts with these keys populated
232 // clear the shortcuts for Backspace and Shift+Backspace to go back or
233 // forward respectively.
234 delete shortcutDataCache['backspace'];
235 delete shortcutDataCache['backspace<>SHIFT'];
236 }
237 232
238 return shortcutDataCache; 233 return shortcutDataCache;
239 } 234 }
240 235
241 /** 236 /**
242 * Returns the keyboard overlay ID. 237 * Returns the keyboard overlay ID.
243 * @return {string} Keyboard overlay ID. 238 * @return {string} Keyboard overlay ID.
244 */ 239 */
245 function getKeyboardOverlayId() { 240 function getKeyboardOverlayId() {
246 return keyboardOverlayId; 241 return keyboardOverlayId;
(...skipping 705 matching lines...) Expand 10 before | Expand all | Expand 10 after
952 function onWidgetShown() { 947 function onWidgetShown() {
953 setTimeout(function() { 948 setTimeout(function() {
954 // Show and focus the instructions div after a delay so that chrome vox 949 // Show and focus the instructions div after a delay so that chrome vox
955 // speaks it correctly. 950 // speaks it correctly.
956 $('instructions').style.visibility = 'visible'; 951 $('instructions').style.visibility = 'visible';
957 $('instructions').focus(); 952 $('instructions').focus();
958 }, 500); 953 }, 500);
959 } 954 }
960 955
961 document.addEventListener('DOMContentLoaded', init); 956 document.addEventListener('DOMContentLoaded', init);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698