bad call to this._reset()

RESOLVED FIXED

Status

Fennec Graveyard
General
P1
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: bcombee, Assigned: bcombee)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Created attachment 384420 [details] [diff] [review]
Remove this.reset() call
Attachment #384420 - Flags: review?(mark.finkle)
(Assignee)

Comment 1

8 years ago
This call was mistakenly included in the patch to fix endPanning... I had it fixed locally, but didn't update the diff in the bug.
(Assignee)

Comment 2

8 years ago
Created attachment 384427 [details] [diff] [review]
Also add this.reset() missing from endKinetic

Also found missing this.reset() call from endKinetic... I think this was a misapply from a patch reject.  I'd fixed both of these locally before.
Attachment #384420 - Attachment is obsolete: true
Attachment #384427 - Flags: review?(mark.finkle)
Attachment #384420 - Flags: review?(mark.finkle)
(Assignee)

Updated

8 years ago
Assignee: nobody → combee
Status: NEW → ASSIGNED
Priority: -- → P1
Attachment #384427 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mobile-browser/rev/710ad9ef9da6
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.