Move controller.doubleClick to EventUtils.js

VERIFIED FIXED

Status

Testing Graveyard
Mozmill
VERIFIED FIXED
9 years ago
a year ago

People

(Reporter: whimboo, Unassigned)

Tracking

Details

(Whiteboard: [verified-mozmill-1.2])

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
Created attachment 387416 [details] [diff] [review]
Patch v1

Now that controller.click has been moved to use the synthesizeMouse function we should do the same for controller.doubleClick.

You can find the patch attached.
Attachment #387416 - Flags: review?(mrogers)
(Reporter)

Comment 1

9 years ago
We should take it into account for Mozmill 1.2.
Whiteboard: [mozmill-1.2]
(Reporter)

Comment 2

9 years ago
(In reply to comment #1)
> We should take it into account for Mozmill 1.2.

Otherwise it will not work and at least one test cannot be automated.
(Reporter)

Updated

9 years ago
Blocks: 500542
Committed in r526.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

9 years ago
Verified fixed with the newTab test where we double click the empty tab strip.

http://code.google.com/p/mozmill/source/detail?r=526
Status: RESOLVED → VERIFIED
Whiteboard: [mozmill-1.2] → [verified-mozmill-1.2]
(Reporter)

Updated

3 years ago
Attachment #387416 - Flags: review?(mikeal.rogers)
(Assignee)

Updated

a year ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.