browser_tapping.js forgets one time to removeEventListener panFinished

RESOLVED FIXED in Firefox 9

Status

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: martijn.martijn, Assigned: martijn.martijn)

Tracking

Trunk
Firefox 9

Details

(Whiteboard: QA-)

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 557690 [details] [diff] [review]
patch

I noticed a js error in one of the following tests coming from browser_tapping.js.
Then I noticed that there is one case in that file, where one addEventListener wasn't removed.
Attachment #557690 - Flags: review?(mbrubeck)
Attachment #557690 - Flags: review?(mbrubeck) → review+
(Assignee)

Updated

7 years ago
Keywords: checkin-needed

Comment 1

7 years ago
Similar to bug 681640 comment 6, please can you fix your hgrc and add a commit message, since my doing so for half a dozen checkin-neededs a day takes a not-insignificant amount of time (also as it happens m-i is closed currently, so can't push it immediately anyway). Thanks :-)
Assignee: nobody → martijn.martijn
Status: NEW → ASSIGNED
Keywords: checkin-needed
OS: Windows 7 → All
Hardware: x86 → All
(Assignee)

Updated

7 years ago
Keywords: checkin-needed

Comment 2

7 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/c4d9981dc228
Keywords: checkin-needed
Whiteboard: [inbound]
(In reply to Jacek Caban from comment #2)
> http://hg.mozilla.org/integration/mozilla-inbound/rev/c4d9981dc228

landed in m-c
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 9

Updated

7 years ago
Whiteboard: QA?

Updated

7 years ago
Whiteboard: QA? → QA-
You need to log in before you can comment on or make changes to this bug.