Bug 33783 – [DRT][Mac] Add modifiers parameter to mouseDown() and mouseUp()

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Kent Tamura]

Add support for mouse events with key modifiers like:
  eventSender.mouseDown(0, ["addSelectionKey"])
  eventSender.mouseUp(0, ["shiftKey", "ctrlKey"])

[Kent Tamura]

Created attachment 46790 [details]
Patch

[Darin Adler]

Comment on attachment 46790 [details]
Patch

> -    [self performSelector:@selector(mouseDown:) withObject:nil afterDelay:0];
> -    [self performSelector:@selector(mouseUp:) withObject:nil afterDelay:0];
> +    [self performSelector:@selector(mouseDown:withModifiers:) withObject:nil afterDelay:0];
> +    [self performSelector:@selector(mouseUp:withModifiers:) withObject:nil afterDelay:0];

This isn't reliable. The argument to "withModifiers" will be random data on the stack. The object "nil" is passed as the first argument, but no second argument is passed. The simplest way I can think of to make this work is to keep a method with the old name and have it call the new method. There may be other alternatives.

review- because of this

[Kent Tamura]

Created attachment 46876 [details]
Proposed patch (rev.2)

[Kent Tamura]

(In reply to comment #2)
> This isn't reliable. The argument to "withModifiers" will be random data on the
> stack. The object "nil" is passed as the first argument, but no second argument
> is passed. The simplest way I can think of to make this work is to keep a
> method with the old name and have it call the new method. There may be other
> alternatives.

Thank you for the advice.  I did so in the updated patch.

[Darin Adler]

Comment on attachment 46876 [details]
Proposed patch (rev.2)

Looks fine.

r=me

Not directly related to this patch: I realize now that the use of performSelector:withObject: is already wrong for mouseDown: and mouseUp: since they have an argument that is not an object. Someone should clean this up in the future. I think we can just replace mouseDown: and mouseUp: with mouseDownButton0 and mouseUpButton0 methods that don't take arguments at all.

[WebKit Commit Bot]

Comment on attachment 46876 [details]
Proposed patch (rev.2)

Clearing flags on attachment: 46876

Committed r53498: <http://trac.webkit.org/changeset/53498>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.