1
edit
Changes
→Remaining Test Reviews: Added peer review for test_MouseEvents.html
|X
|
|X
|
|
|northWind
|
|'''FIXEDPEER REVIEWED, AWAITING PEER REVIEWSEE COMMENTS'''
|
|
* northWind:
** hangs intermittently, most likely requires focus, see [[#Waiting For Focus|here]]
** 39,41: not required, handled by harness
** 51: remove line, setup being called at better time @343
** 60: should use === for identity as opposed to ==
** 60: should add document.mozFullScreen && to condition
** 61, 69: needs failure callback with is(false, ...) and SimpleTest.finish()
** 68: should add && document.mozFullScreenElement === outer to condition
** 68: whitespace at end of line
** 78: looked at this briefly, are we sure that synthMs will cause the cursor to cross over the 'one' div?
** 172: Col 45: "the" should be "a Left"
** 183-214: Why not testing for mousemove?
|-
|116
