RESOLVED WONTFIX 49607
[Chromium] Need unit tests for WebInputEventFactory 
https://bugs.webkit.org/show_bug.cgi?id=49607
Summary [Chromium] Need unit tests for WebInputEventFactory
Stuart Morgan
Reported 2010-11-16 11:27:54 PST
The OS event conversions that happen in WebInputEventFactory don't appear to have any unit tests, and the event tests in general (from what I could find) generate internal event structures and run tests based on those. There doesn't seem to be any testing of the actual OS event conversion, which isn't ideal.
Attachments
Add attachment proposed patch, testcase, etc.
chandra shekar vallala
Comment 1 2011-10-19 05:30:26 PDT
(In reply to comment #0) > The OS event conversions that happen in WebInputEventFactory don't appear to have any unit tests, and the event tests in general (from what I could find) generate internal event structures and run tests based on those. There doesn't seem to be any testing of the actual OS event conversion, which isn't ideal. "testing of the actual OS event conversion" means should we check for all WebInputEvent values ?
Stuart Morgan
Comment 2 2011-10-19 05:46:28 PDT
Meaning we should start with a native OS event with known properties, run it through the conversion, and ensure that the resulting WebInputEvent has those properties. (As an example, I filed this bug after finding and fixing a bug in the click count never being set when converting a Mac mouse event.)
Note You need to log in before you can comment on or make changes to this bug.