Add a test to ensure the following: 1. When a person presses e, g and releases e, g (in that order) that the DOM keydown and keyup events are dispatches in the same order as the key were pressed and released. 2. Ensure (1) when the keys are left Option and right Option.
Created attachment 375844 [details] Patch
Comment on attachment 375844 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=375844&action=review > LayoutTests/platform/ios/TestExpectations:3361 > +# Skip until we have the fixes for <rdar://problem/53613454> and <rdar://problem/54001139>. > +fast/events/ios/multiple-key-press-and-release-ordering.html [ Skip ] Why not expect a failure instead of skipping?
Because its failure mode is a time out 😀. I had two options because of that: 1. Make test wait minimum amount of time before declaring failure 2. Mark the test as timeout in expectations Both options have disadvantages though I was really leaning towards (2) because it's not extra work for me. I chose not to do (2) because someone once told me not to because "machines have limited resources" reasons. Not sure if that is true anymore.😕
Skip seems fine for a timeout that will be fixed soon.
Comment on attachment 375844 [details] Patch Clearing flags on attachment: 375844 Committed r248531: <https://trac.webkit.org/changeset/248531>
All reviewed patches have been landed. Closing bug.
<rdar://problem/54214292>