Closed
Bug 629257
Opened 14 years ago
Closed 14 years ago
Mochitest for bug 598375
Categories
(Firefox Graveyard :: Panorama, defect, P4)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 5
People
(Reporter: iangilman, Assigned: ttaubert)
References
Details
(Whiteboard: [bugday-20110513] )
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
Tim had a good idea (in the thread for bug 598375) for how to test that bug:
> I think we could use EventUtils.synthesizeMouseExpectEvent() to fire over the
> search button and see if the right element gets the event?!
Now we just need to write it.
|
Assignee | |
Comment 1•14 years ago
|
||
I recently tried to use EventUtils.synthesizeMouseExpectEvent() and it's not yet ready to be used in panorama tests.
Seems "is()" is not defined - https://mxr.mozilla.org/mozilla-central/source/testing/mochitest/tests/SimpleTest/EventUtils.js#467
|
||
Comment 2•14 years ago
|
||
bugspam
|
|
Assignee | |
Updated•14 years ago
|
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 3•14 years ago
|
||
Attachment #524305 -
Flags: feedback?(raymond)
|
||
Updated•14 years ago
|
Attachment #524305 -
Flags: feedback?(raymond) → feedback+
|
|
Assignee | |
Comment 4•14 years ago
|
||
Comment on attachment 524305 [details] [diff] [review]
patch v1
Passed try:
http://tbpl.mozilla.org/?tree=MozillaTry&pusher=tim.taubert@gmx.de&rev=947c561a6738
Attachment #524305 -
Flags: review?(ian)
|
Reporter | |
Comment 5•14 years ago
|
||
Comment on attachment 524305 [details] [diff] [review]
patch v1
nice :)
Attachment #524305 -
Flags: review?(ian) → review+
|
|
Assignee | |
Comment 6•14 years ago
|
||
Attachment #524305 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•14 years ago
|
Keywords: checkin-needed
|
||
Comment 7•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox4.2
|
||
Updated•14 years ago
|
Target Milestone: Firefox5 → Firefox 5
|
||
Comment 8•13 years ago
|
||
Check to see if this is safe to mark verified. Is this test passing?
|
|
Assignee | |
Comment 9•13 years ago
|
||
(In reply to comment #8)
> Check to see if this is safe to mark verified. Is this test passing?
Yep, we would have re-opened this bug otherwise :)
|
|
||
Updated•13 years ago
|
Whiteboard: [bugday-20110513]
|
||
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•