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

Unified Diff: sky/tests/services/event-sender.sky

Issue 872233002: Switch KeyboardEvents over to NewEventHandler (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: nits Created 5 years, 11 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: sky/tests/services/event-sender.sky
diff --git a/sky/tests/services/event-sender.sky b/sky/tests/services/event-sender.sky
index 99472060aaf26376aa80bfb81dbe687245df3614..99e15cbff32b1fe23a11d7fce9788efda3718dc4 100644
--- a/sky/tests/services/event-sender.sky
+++ b/sky/tests/services/event-sender.sky
@@ -10,10 +10,10 @@
describe('Sky event sender', function() {
it('should be able to send events', function(done) {
var sky = document.querySelector('sky')
- sky.focus();
sky.addEventListener('keypress', function(event) {
assert.equal(event.type, 'keypress');
- assert.equal(event.keyCode, 0x41);
+ assert.equal(event.virtualKeyCode, 0);
+ assert.equal(event.charCode, 0x41);
done();
});
@@ -22,7 +22,7 @@ describe('Sky event sender', function() {
testHarness.dispatchInputEvent(new events.Event({
action: constants.EventType.KEY_PRESSED,
key_data: new events.KeyData({
- native_key_code: 0x41,
+ windows_key_code: 0x41,
text: 0x41,
unmodified_text: 0x41,
is_char: true,

Powered by Google App Engine
This is Rietveld 408576698