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

Unified Diff: chrome/test/data/instant_extended.html

Issue 11889003: Fixing ESC in instant-extended. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Reworked to send ESC down to JS, added test. Created 7 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/instant_extended.html
diff --git a/chrome/test/data/instant_extended.html b/chrome/test/data/instant_extended.html
index 96c2541b5528f4e9a0c5587c3428c669a2e5cdb9..413d5c86db2aa9de247d535a86fab8e317611878 100644
--- a/chrome/test/data/instant_extended.html
+++ b/chrome/test/data/instant_extended.html
@@ -49,21 +49,29 @@ function previousSuggestion() {
if (suggestionIndex != -1) {
suggestionIndex--;
if (suggestionIndex == -1) {
- apiHandle.setValue(savedUserText);
+ resetSuggestion();
} else {
apiHandle.setValue(suggestions[suggestionIndex]);
}
}
}
+function resetSuggestion() {
+ apiHandle.setValue(savedUserText);
+ suggestionIndex = -1;
+}
+
function handleKeyPress(event) {
var VKEY_UP = 0x26;
var VKEY_DOWN = 0x28;
+ var VKEY_ESCAPE = 0x1B;
if (event.keyCode == VKEY_DOWN) {
nextSuggestion();
} else if (event.keyCode == VKEY_UP) {
previousSuggestion();
+ } else if (event.keyCode == VKEY_ESCAPE) {
+ resetSuggestion();
}
}

Powered by Google App Engine
This is Rietveld 408576698