Closed
Bug 859939
Opened 12 years ago
Closed 12 years ago
AsyncPanZoomController has some type wrongness
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla23
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
|
1.75 KB,
patch
|
ajones
:
review+
|
Details | Diff | Splinter Review |
mLastEventTime is assigned to from a uint32_t value that is potentially outside the range of the int32_t, so mLastEventTime should be a uint32_t as well. I think practically this doesn't make a difference because when it is used it always get casted to a uint32_t anyway but while debugging it was displaying a negative value which was annoying.
Also mPos is an int not a float.
Attachment #735311 -
Flags: review?(ajones)
|
||
Updated•12 years ago
|
Attachment #735311 -
Flags: review?(ajones) → review+
|
Assignee | |
Comment 1•12 years ago
|
||
|
||
Comment 2•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in
before you can comment on or make changes to this bug.
Description
•