Closed Bug 1395168 Opened 7 years ago Closed 7 years ago

Add a tests for key synthesis

Categories

(Firefox for Android Graveyard :: Keyboards and IME, enhancement)

All
Android
enhancement
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed

People

(Reporter: jchen, Assigned: jchen)

References

Details

Attachments

(1 file)

Add tests for when we synthesize key events, including a test for bug 1387889.
Add tests for synthesizing keys, including test for dummy keys and test for wrong metastate for synthesized non-English keys (i.e. bug 1387889).
Attachment #8902705 - Flags: review?(esawin)
Comment on attachment 8902705 [details] [diff] [review] Add tests for key synthesis (v1) Review of attachment 8902705 [details] [diff] [review]: ----------------------------------------------------------------- ::: mobile/android/tests/browser/robocop/robocop_input.html @@ +82,5 @@ > + case "keydown": > + case "keypress": > + case "keyup": > + key_log += event.type + ":" + event.key + "," + > + get_key_metastate(event) + ";"; Template-strings might be nice for this. @@ +85,5 @@ > + key_log += event.type + ":" + event.key + "," + > + get_key_metastate(event) + ";"; > + break; > + default: > + key_log += "unknown;" Missing ;.
Attachment #8902705 - Flags: review?(esawin) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: